[PATCH] D28907: [SLP] Fix for PR30787: Failure to beneficially vectorize 'copyable' elements in integer binary ops.

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:469
+    return InstructionsState(VL[BaseIndex], cast<Instruction>(VL[BaseIndex]),
+                             cast<Instruction>(VL[AltIndex]), IsNonAlt);
+  else
----------------
InstructionState was keeping the Base/Alt instructions the same if AltOpcodeNum ==0 (BaseIndex ==AltIndex) - why are you inserting nulls?


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:1526
 
+static Value *getDefaultConstantForOpcode(unsigned Opcode, Type *Ty) {
+  switch(Opcode) {
----------------
Can we use ConstantExpr::getBinOpIdentity instead?


https://reviews.llvm.org/D28907