[PATCH] D38696: [DAGCombine] Permit combining of shuffle of equivalent splat BUILD_VECTORs

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:15492-15493
 
+  // If both inputs are splats of the same value then we can safely merge this
+  // to a single BUILD_VECTOR with only the common undef elements.
+  auto *BV0 = dyn_cast<BuildVectorSDNode>(N0);
----------------
Is this more conservative than necessary? If the shuffle mask is choosing an undef lane from either input, then the output would still be undef?


Repository:
  rL LLVM

https://reviews.llvm.org/D38696