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

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Seems like a good idea to me too. A question on the "fusion" inlined.



================
Comment at: llvm/lib/Target/ARM/ARMSubtarget.h:708
   /// Return true if the CPU supports any kind of instruction fusion.
-  bool hasFusion() const { return hasFuseAES() || hasFuseLiterals(); }
+  bool hasFusion() const { return hasFuseAES() || hasFuseLiterals() || hasLOB(); }
 
----------------
Do we need to add LOB here, i.e. are we using that?
If so, is "fusing" the right thing? Is the use of LOB here the same as e.g. "fusing AES" instructions?


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

https://reviews.llvm.org/D79767