[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 08:40:58 PST 2023


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

LG with a nit



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7593
       // Debug information does not impact spill cost.
-      if ((isa<CallInst>(&*PrevInstIt) &&
-           !isa<DbgInfoIntrinsic>(&*PrevInstIt)) &&
+      if ((isa<CallInst>(&*PrevInstIt)) && !NoCallIntrinsic(&*PrevInstIt) &&
           &*PrevInstIt != PrevInst)
----------------
Remove extra parens around isa call.


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

https://reviews.llvm.org/D140392



More information about the llvm-commits mailing list