This is an archive of the discontinued LLVM Phabricator instance.

[RDA] Add def checks to isSafeToRemove
AbandonedPublic

Authored by samparker on Sep 28 2020, 7:42 AM.

Details

Summary

We were tracking a use chain from a def, but we weren't considering any register defs that weren't a part of the chain. This could result in the program being changed incorrectly. This has resulted in a few ARMLowOverheadLoop test changes around the loop iteration copies because that has also been changed to more accurately query the safety of removing LoopDec.

Diff Detail

Event Timeline

samparker created this revision.Sep 28 2020, 7:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 28 2020, 7:42 AM
samparker requested review of this revision.Sep 28 2020, 7:42 AM
samparker abandoned this revision.Sep 29 2020, 8:08 AM

I need to break this up and change it.