[PATCH] D88209: [ARM] Check for LSTP side-effects.

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 05:13:10 PDT 2020


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Thanks, perfectly clear, LGTM.

For bonus points, if you have ideas on that, perhaps a TODO how we can win some of the reverted loops back? Or is it next on your list to look at this?. From a quick look at one of the changed tests, I could be wrong, I see some loopstart instructions sitting in the middle of the block for no good reasons? Is there some low hanging fruit there?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88209/new/

https://reviews.llvm.org/D88209



More information about the llvm-commits mailing list