[PATCH] Make DAGCombiner work on vector bitshifts with constant splat vectors.

Tim Northover t.p.northover at gmail.com
Fri Mar 7 10:17:37 PST 2014


  Hmm, I'm still not seeing it I'm afraid:


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:3983-3984
@@ +3982,4 @@
+        return DAG.getConstant(0, VT);
+      return DAG.getNode(ISD::SHL, SDLoc(N), VT, N0.getOperand(0),
+                         DAG.getConstant(c1 + c2, N1.getValueType()));
+    }
----------------
Here, for example: there's no BUILD_VECTOR so if VT is a vector type why don't we end up with (shl x, CONSTANT)?


http://llvm-reviews.chandlerc.com/D2997



More information about the llvm-commits mailing list