[PATCH] D81766: [VectorCombine] try to create vector loads from scalar loads

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D81766#2202770 <https://reviews.llvm.org/D81766#2202770>, @RKSimon wrote:

> LGTM with one minor @nikic Does this look OK now?

Looks good to me as well.



================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:108
+  if (!PtrOpTy)
+    return false;
+
----------------
assert / use `cast<>`? Don't think the pointer operand can have a non-pointer type.


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

https://reviews.llvm.org/D81766