This is an archive of the discontinued LLVM Phabricator instance.

[ARM][LowOverheadLoops] Remove modifications to the correct element count register
ClosedPublic

Authored by samtebbs on Sep 7 2020, 7:55 AM.

Details

Summary

After my patch in D86087, code that now uses the mov operand rather than the vctp operand will no longer remove modifications to the vctp operand as they should. This patch fixes that by explicitly removing modifications to the vctp operand rather than the register used as the element count.

Diff Detail

Event Timeline

samtebbs created this revision.Sep 7 2020, 7:55 AM
samtebbs requested review of this revision.Sep 7 2020, 7:55 AM
samtebbs edited the summary of this revision. (Show Details)
samparker accepted this revision.Sep 8 2020, 12:39 AM

Easy mistake to have made. Thanks, LGTM

This revision is now accepted and ready to land.Sep 8 2020, 12:39 AM