This is an archive of the discontinued LLVM Phabricator instance.

[SystemZ] Define MaxInstLength to have the value of 6.
ClosedPublic

Authored by jonpa on Oct 23 2020, 11:51 AM.

Details

Summary

This value had the default value of 4 which caused branch relaxation to fail.

Note: The python tests show up as UNSUPPORTED when I try to run them with llvm-lit. I could not find out how to enable them. The test case I made takes 1.2 secs to execute on a fast machine. Perhaps it could be faster with a huge .mir test starting after regalloc...?

Diff Detail

Event Timeline

jonpa created this revision.Oct 23 2020, 11:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 23 2020, 11:51 AM
jonpa requested review of this revision.Oct 23 2020, 11:51 AM
uweigand accepted this revision.Oct 23 2020, 1:53 PM

LGTM.

Note: The python tests show up as UNSUPPORTED when I try to run them with llvm-lit. I could not find out how to enable them.

They are enabled by using the following cmake argument:

-DLLVM_LIT_ARGS='-sv --param run_long_tests=true'
This revision is now accepted and ready to land.Oct 23 2020, 1:53 PM
This revision was automatically updated to reflect the committed changes.