The test case is full of windows line endings.<br><br>+      // VPERMD<br>+      int ShufMask[] = {0, 2, 4, 6, -1, -1, -1, -1};<br><br>That should probably be "static const int ShufMask"<br><br><br>+      int ShufMask[] = {0,  2,  -1,  -1};<br>
<br>Same here. And only use 1 space after the comma.<br><br><br>+      Op = DAG.getNode(X86ISD::PSHUFB, dl, MVT::v32i8, Op, BV);<br>+         <br>+      Op = DAG.getNode(ISD::BITCAST, dl, MVT::v4i64, Op);<br>+<br>+      int ShufMask[] = {0,  2,  -1,  -1};<br>
+      Op = DAG.getVectorShuffle(MVT::v4i64, dl,  Op, DAG.getUNDEF(MVT::v4i64), &ShufMask[0]);<br>+<br>+      Op =  DAG.getNode(ISD::EXTRACT_SUBVECTOR, dl, MVT::v2i64, Op, DAG.getIntPtrConstant(0));<br>+   <br>+      return DAG.getNode(ISD::BITCAST, dl, VT, Op);<br>
<br>There are some dangling spaces in there. Also need to fit in 80 columns<br><br><br>+      SDValue pshufbConstPool = DAG.getNode(ISD::BUILD_VECTOR, dl,<br>+                                 MVT::v32i8, &pshufbMask[0], 32);<br>
<br>This looks unused<br><br><br>     if (((VT == MVT::v8i32) && (OpVT == MVT::v8i16))  ||<br>       ((VT == MVT::v4i64) && (OpVT == MVT::v4i32)))  {<br><br>This isn't your code, but can you fix the alignment so that the second line starts after the first line's opening parenthese.<br>
<br><br>+let Predicates = [HasAVX2] in {<br>+  let AddedComplexity = 15 in {<br>+    def : Pat<(v4i64 (X86vzmovly (v4i32 VR128:$src))), (VPMOVZXDQYrr VR128:$src)>;<br>+    def : Pat<(v8i32 (X86vzmovly (v8i16 VR128:$src))), (VPMOVZXWDYrr VR128:$src)>;<br>
+  }<br>+}<br><br>Why the AddedComplexity? Also 80 columns.<br><br><br>+      AddToWorkList(Op.getNode());<br>     } else if (Op.getValueType().bitsGT(VT)) {<br>       Op = DAG.getNode(ISD::TRUNCATE, N->getDebugLoc(), VT, Op);<br>
-    }<br>+      AddToWorkList(Op.getNode());<br>+   }<br><br>Last bracket needs to be right 1 space.<br><br><br><div class="gmail_quote">On Thu, Apr 19, 2012 at 10:50 AM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div class="im">On Thu, Apr 19, 2012 at 10:29 AM, Chad Rosier <span dir="ltr"><<a href="mailto:mcrosier@apple.com" target="_blank">mcrosier@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Mostly stylistic comments:<br>
<br>
+      SmallVector<SDValue,32> pshufbMask;<br>
+      for (unsigned i=0; i<2; i++) {<br>
<br>
Spacing: for (unsigned i = 0; i < 2; i++)<br></blockquote></div><div><br>That should also be "++i" to match coding standards.<br> <br>I'll try to review the content tonight.<span class="HOEnZb"><font color="#888888"><br>
</font></span></div></div><span class="HOEnZb"><font color="#888888"><br>-- <br>~Craig<br>

</font></span></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>