This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC][AIX] Fallback to DAG-ISEL if a global has toc-data attribute.
ClosedPublic

Authored by sfertile on Dec 8 2021, 12:04 PM.

Details

Summary

FAST-ISEL should fall back to DAG-ISEL when a global variable has the toc-data attribute. A number of the 64-bit checks were
duplicated in the lit test because of

  1. Slightly different output between -O0 and -O2 due to FAST-ISEL vs DAG-ISEL codegen.
  2. In preperation of a peephole optimization that will run when optimizations are enabled.

Also updates where we dump the mir since the ppc-ctr-loops-verify pass happens at a different time in the pipeline at -O0 when generating 32-bit code. I

Diff Detail

Event Timeline

sfertile created this revision.Dec 8 2021, 12:04 PM
sfertile requested review of this revision.Dec 8 2021, 12:04 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 8 2021, 12:04 PM
sfertile edited reviewers, added: yoalione; removed: yousufhali.Dec 8 2021, 12:07 PM
sfertile updated this revision to Diff 395489.Dec 20 2021, 10:02 AM
  • Fix if condition that broke the Debian buildbot.
  • Remove 'Required: asserts' from the test as I've updated to use a pass that is run regardless of whether asserts are enabled or not.
  • Remove some unintended formatting changes from the lit test.
This revision is now accepted and ready to land.Dec 21 2021, 9:03 AM
This revision was landed with ongoing or failed builds.Jan 17 2022, 1:24 PM
This revision was automatically updated to reflect the committed changes.