[PATCH] D140392: [SLP][AArch64] Incorrectly estimated intrinsic as a function call

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 3 05:30:20 PST 2023


ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7572-7574
+          if (II->isAssumeLikeIntrinsic()) {
+            return true;
+          } else {
----------------
You can drop braces here


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7574
+            return true;
+          } else {
+            FastMathFlags FMF;
----------------
No need for else after if with the return substatemenet.


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

https://reviews.llvm.org/D140392



More information about the llvm-commits mailing list