[PATCH] Allow sret on the second parameter as well as the first

David Majnemer david.majnemer at gmail.com
Fri May 9 11:47:44 PDT 2014


LGTM

================
Comment at: lib/Target/X86/X86ISelLowering.cpp:2310
@@ -2306,1 +2309,3 @@
+        break;
+    SDValue Copy = DAG.getCopyToReg(DAG.getEntryNode(), dl, Reg, InVals[I]);
     Chain = DAG.getNode(ISD::TokenFactor, dl, MVT::Other, Copy, Chain);
----------------
We should assert `I != E`.

http://reviews.llvm.org/D3617






More information about the llvm-commits mailing list