[PATCH] D79767: [ARM] Macro fuse t2LoopDec and t2LoopEnd

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

In D79767#2031292 <https://reviews.llvm.org/D79767#2031292>, @samparker wrote:

> Seems like a good idea to me, I can't see this causing any harm. but do you have an example of where it enables LOBs?


Not where it's helping LOB's, exactly. It can help keep the live range of cpsr shorter, allowing us to use more t1 instructions, which can help a little. I've not seen it improve the issue with spilling LR between the t2LoopDec and t2LoopEnd. I think the problem there might be that there is only really a single place for the value to spill at, between those two instructions. Depending on how we try to fix that, this might be useful in it's own right or not.


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

https://reviews.llvm.org/D79767