[PATCH] D68499: [PATCH 16/38] [noalias] Loop vectorizer: learn about noalias intrinsics

[Jeroen Dobbelaere via Phabricator via llvm-commits]

jeroen.dobbelaere marked 2 inline comments as done.
jeroen.dobbelaere added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:3223-3227
+  // Compute corresponding vector type for return value and arguments.
+  Type *RetTy = ToVectorTy(ScalarRetTy, VF);
+  for (Type *ScalarTy : ScalarTys)
+    Tys.push_back(ToVectorTy(ScalarTy, VF));
+
----------------
lebedev.ri wrote:
> Why was this moved? Just as an optimization in case we early-return inbetween?
By moving this later, we avoid trying to get the vector variant of MetadataAsValue for the noalias intrinsics.
(and also the corresponding assertion)


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

https://reviews.llvm.org/D68499