[PATCH] D82550: [SLPVectorizer] handle vectorized lib functions

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D82550#2130742 <https://reviews.llvm.org/D82550#2130742>, @RKSimon wrote:

> @ABataev any more comments?






================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3030
+
+      auto Shape =
+          VFShape::get(*CI, {static_cast<unsigned int>(VL.size()), false},
----------------
Is this a pointer? Then `auto *`.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:5121
+    if (I->mayReadOrWriteMemory() && !isSideeffectIntrinsic(I) &&
+        !isVectorizableLibFunctionCall(I)) {
       // Update the linked list of memory accessing instructions.
----------------
Why do you need to exclude vectorizable library functions here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82550