This is an archive of the discontinued LLVM Phabricator instance.

XFAIL external resources test cases on big-endian systems
ClosedPublic

Authored by uweigand on Jul 11 2022, 6:10 AM.

Details

Summary

Since https://reviews.llvm.org/D126446 was merged, the builder on s390x is red, as parsing external resources does not work correctly on big-endian machines. To make the CI green again, this patch:

  • Always reads and writes the "alignment" tag in little-endian byte order (this fixes failing assertions)
  • XFAIL the test case that is still failing after that fix

Diff Detail

Event Timeline

uweigand created this revision.Jul 11 2022, 6:10 AM
uweigand requested review of this revision.Jul 11 2022, 6:10 AM

Can you split the unrelated change in a separate commit?

Feel free to XFAIL for your bit without a review.

Can you split the unrelated change in a separate commit?

Feel free to XFAIL for your bit without a review.

+1, LGTM otherwise

This revision was not accepted when it landed; it landed in state Needs Review.Jul 12 2022, 12:37 AM
This revision was automatically updated to reflect the committed changes.

I've now committed both changes in separate commits. Thanks!

SystemZ mlir builder is now back to green. Of course, the underlying issue still needs to be fixed to we can remove the XFAIL again.