[PATCH] D57779: [SLP] Add support for throttling.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 08:42:09 PDT 2020


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7269
+                 << " and threshold "
+                 << ore::NV("Threshold", -SLPCostThreshold);
+        });
----------------
This looks like a NFC clang-format change now - either pre-commit or discard from the patch?


================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/load-merge.ll:59
 ; CHECK-NEXT:    [[TMP2:%.*]] = load <2 x float>, <2 x float>* [[TMP1]], align 4
-; CHECK-NEXT:    [[X2:%.*]] = load float, float* [[GEP2]]
+; CHECK-NEXT:    [[X2:%.*]] = load float, float* [[GEP2]], align 4
 ; CHECK-NEXT:    [[TMP3:%.*]] = extractelement <2 x float> [[TMP2]], i32 0
----------------
rebase - this was committed at rG90f721404ff8


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

https://reviews.llvm.org/D57779



More information about the llvm-commits mailing list