[PATCH] D144128: [SLP] Check with target before vectorizing GEP Indices

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa marked an inline comment as done.
jonpa added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:11605-11612
+      continue;
     }
 
     // Ignore getelementptr instructions that have more than one index, a
     // constant index, or a pointer operand that doesn't point to a scalar
     // type.
+    if (TTI->prefersVectorizedAddressing())
----------------
ABataev wrote:
> Could you try to remove this change and keep just the change in NotProfitableForVectorization?
sorry - yeah I tried that, but then those changes dissapeared, so it seems that these two points do not overlap.


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

https://reviews.llvm.org/D144128