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

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 00:27:46 PDT 2019


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

Looks very reasonable to me.



================
Comment at: lib/Target/ARM/ARMTargetTransformInfo.cpp:815
+      case Intrinsic::loop_decrement_reg:
+        LLVM_DEBUG(dbgs() << "Found intrinsic.\n");
+        return true;
----------------
nitpick: perhaps "found hwloop intrinsic"


================
Comment at: test/Transforms/HardwareLoops/ARM/structure.ll:138
+
+; Function Attrs: noduplicate nounwind
+declare void @llvm.set.loop.iterations.i32(i32) #0
----------------
another nit: don't need the comments


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

https://reviews.llvm.org/D63212





More information about the llvm-commits mailing list