[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