[PATCH] D87457: [ARM][TTI] Prevents constants in a min(max) or max(min) pattern from being hoisted when in a loop

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 21 10:03:19 PDT 2020


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

Very nice, LGTM.



================
Comment at: llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp:297
+
+    auto isSSatMin = [=](Value *MinInst) {
+      if (isa<SelectInst>(MinInst)) {
----------------
This can use [&]
(it likely doesn't matter a lot, but can prevent a copy).


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

https://reviews.llvm.org/D87457



More information about the llvm-commits mailing list