This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Remove DLS lr, lr
ClosedPublic

Authored by dmgreen on Apr 27 2020, 4:51 AM.

Details

Summary

A DLS lr, lr only move lr to itself, and can be removed as a result.

Diff Detail

Event Timeline

dmgreen created this revision.Apr 27 2020, 4:51 AM
samparker added inline comments.Apr 27 2020, 5:29 AM
llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp
1297

Would be nicer to handle this in ExpandLoopStart.

dmgreen updated this revision to Diff 260319.Apr 27 2020, 7:15 AM

Now in ExpandStart

Cheers, LGTM.

samparker accepted this revision.May 5 2020, 11:49 PM
This revision is now accepted and ready to land.May 5 2020, 11:49 PM
This revision was automatically updated to reflect the committed changes.