This is an archive of the discontinued LLVM Phabricator instance.

[z/OS] Clean up some z/OS testcases.
Needs ReviewPublic

Authored by Everybody0523 on Mar 24 2022, 12:01 PM.

Details

Reviewers
Kai
uweigand
jonpa
Summary

This patch does some cleanup on several z/OS testcases. In particular, CHECK is replaced with CHECK-DAG in many places where the exact ordering of the instructions checked for does not matter. This allows for avoiding test failures in semi-related changes.

Diff Detail

Unit TestsFailed

Event Timeline

Everybody0523 created this revision.Mar 24 2022, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2022, 12:01 PM
Everybody0523 requested review of this revision.Mar 24 2022, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2022, 12:01 PM
uweigand added inline comments.Apr 26 2022, 6:59 AM
llvm/test/CodeGen/SystemZ/call-zos-vararg.ll
17

This may be a bit too flexible. The CHECK-DAG here would allow the two instructions above to be swapped, in which case the semantics would be incorrect.

The same holds for a number of other tests below. Maybe a better way would be to use auto-generated CHECK statements for this file? (Via utils/update_llc_test_checks.py)