[PATCH] D63212: [ARM] Scan for existing loop intrinsics

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:839
   HWLoopInfo.CounterInReg = true;
+  HWLoopInfo.IsNestingLegal = false;
   HWLoopInfo.CountType = Type::getInt32Ty(C);
----------------
We can nest hwloops, so it would be legal, but we think it will never be benecial because it involves spilling and reloading LR. Do we really need to set IsNestingLegal? Don't think this is important, but was just curious; I haven't looked and reminded myself how it is used.


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

https://reviews.llvm.org/D63212