This is an archive of the discontinued LLVM Phabricator instance.

[Clang][ARM] Reenable arm_acle.c test.
ClosedPublic

Authored by dmgreen on Mar 12 2021, 7:27 AM.

Details

Summary

This test was apparently disabled in 6fcd4e080f09c9765d6 back in 2019, without any sign of how it was going to be reenabled. This patch rewrites the test to use update_cc_test_checks, with midend optimizations other that mem2reg disabled.

Diff Detail

Event Timeline

dmgreen created this revision.Mar 12 2021, 7:27 AM
dmgreen requested review of this revision.Mar 12 2021, 7:27 AM
vhscampos accepted this revision.Mar 12 2021, 7:34 AM

LGTM. mem2reg is a simple enough optimization to clear up the IR in this case. Thank you.

This revision is now accepted and ready to land.Mar 12 2021, 7:34 AM
This revision was landed with ongoing or failed builds.Mar 14 2021, 3:59 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 14 2021, 3:59 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript