testsuite: Don't disable LLVM ways due to unversioned llc
ClosedPublic

Authored by bgamari on Jan 21 2018, 12:13 PM.

Details

Reviewers
angerman
Summary

Previously we would silently disable the LLVM testsuite ways when LLC is set to
an unversioned llc executable (e.g. llc instead of llc-5.0). Presumably
this was because we were lax in checking the LLVM version in configure.
However, this is no longer the case and consequently this confusing behavior is
no longer necessary.

Test Plan

Validate

bgamari created this revision.Jan 21 2018, 12:13 PM
angerman accepted this revision.Jan 21 2018, 9:09 PM

... and optllvm starts failing ;-)

Unexpected failures:
   /tmp/ghctest-qmrjo3ke/test   spaces/./llvm/should_compile/T5681.run   T5681 [exit code non-0] (optllvm)
   /tmp/ghctest-qmrjo3ke/test   spaces/./llvm/should_compile/T7571.run   T7571 [exit code non-0] (optllvm)
   /tmp/ghctest-qmrjo3ke/test   spaces/./llvm/should_compile/T8131b.run  T8131b [exit code non-0] (optllvm)
   /tmp/ghctest-qmrjo3ke/test spaces/./llvm/should_compile/T11649.run T11649 [exit code non-0] (optllvm)

However, I am supportive of this change. The more generic the better in this case I think.

This revision is now accepted and ready to land.Jan 21 2018, 9:09 PM