It hasn't been applied. Is it okay to apply?<br><br><div class="gmail_quote">On Tue, May 25, 2010 at 10:09 PM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr">baldrick@free.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hi Zhongxing Xu, this looks ok to me.  Was it applied?<br>
<br>
Ciao,<br>
<br>
Duncan.<br>
<div class="im"><br>
> ping?<br>
><br>
> The reason of this change is that in LowerFormalArguments() we have:<br>
><br>
> if (Is64Bit && MF.getFunction()->hasStructRetAttr()) {<br>
>      X86MachineFunctionInfo *FuncInfo =<br>
> MF.getInfo<X86MachineFunctionInfo>();<br>
>      unsigned Reg = FuncInfo->getSRetReturnReg();<br>
>      if (!Reg) {<br>
>        Reg =<br>
> MF.getRegInfo().createVirtualRegister(getRegClassFor(MVT::i64));<br>
>        FuncInfo->setSRetReturnReg(Reg);<br>
>      }<br>
>      SDValue Copy = DAG.getCopyToReg(DAG.getEntryNode(), dl, Reg,<br>
> InVals[0]);<br>
>      Chain = DAG.getNode(ISD::TokenFactor, dl, MVT::Other, Copy, Chain);<br>
>    }<br>
><br>
> On Fri, May 21, 2010 at 3:45 PM, Zhongxing Xu <<a href="mailto:xuzhongxing@gmail.com">xuzhongxing@gmail.com</a><br>
</div><div class="im">> <mailto:<a href="mailto:xuzhongxing@gmail.com">xuzhongxing@gmail.com</a>>> wrote:<br>
><br>
>     Index: lib/Target/X86/X86ISelLowering.cpp<br>
>     ===================================================================<br>
>     --- lib/Target/X86/X86ISelLowering.cpp    (版本 104313)<br>
>     +++ lib/Target/X86/X86ISelLowering.cpp    (工作副本)<br>
>     @@ -1256,10 +1256,8 @@<br>
>           MachineFunction &MF = DAG.getMachineFunction();<br>
>           X86MachineFunctionInfo *FuncInfo =<br>
>     MF.getInfo<X86MachineFunctionInfo>();<br>
>           unsigned Reg = FuncInfo->getSRetReturnReg();<br>
>     -    if (!Reg) {<br>
>     -      Reg = MRI.createVirtualRegister(getRegClassFor(MVT::i64));<br>
>     -      FuncInfo->setSRetReturnReg(Reg);<br>
>     -    }<br>
>     +    assert(Reg &&<br>
>     + "SRetReturnReg should have been set in LowerFormalArguments().");<br>
>           SDValue Val = DAG.getCopyFromReg(Chain, dl, Reg, getPointerTy());<br>
><br>
>           Chain = DAG.getCopyToReg(Chain, dl, X86::RAX, Val, Flag);<br>
><br>
><br>
><br>
><br>
</div><div class="im">> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
</div>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<div class="im"><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
</div><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br>