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

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

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



================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:128
+                                    Load->getPointerAddressSpace());
+  OldCost += TTI.getVectorInstrCost(Instruction::InsertElement, I.getType());
+
----------------
Since you're inserting into undef, this is really a BUILD_VECTOR - you might get better results with getScalarizationOverhead?


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

https://reviews.llvm.org/D81766