This is an archive of the discontinued LLVM Phabricator instance.

[testsuite] Remove testing failures vestiges
ClosedPublic

Authored by davide on Dec 11 2017, 5:07 PM.

Details

Summary

Some tests are failing on macOS when building with the in-tree clang, and this is because they're conditional on the version released.
Apple releases using a different versioning number, but as these are conditional on llvm < 7, they fail for clang ToT (which is 6.0).
As a general solution, we actually need either a mapping between Apple internal release version and public ones, or XFAIL both (which is more cumbersome).

That said, I discussed this with Fred , and Apple Clang 6.0 seems to be old enough that we can remove this altogether (which means I can delay implementing the general purpose solution for a bit).

Thanks!

Diff Detail

Repository
rL LLVM

Event Timeline

davide created this revision.Dec 11 2017, 5:07 PM
davide edited subscribers, added: lldb-commits; removed: llvm-commits.Dec 11 2017, 5:08 PM
vsk accepted this revision.Dec 11 2017, 5:10 PM

Thanks! The 6.0 compiler is ancient at this point. This lgtm.

This revision is now accepted and ready to land.Dec 11 2017, 5:10 PM
This revision was automatically updated to reflect the committed changes.