[llvm-commits] [llvm] r62338 - in /llvm/trunk: lib/Target/X86/X86ISelLowering.cpp test/CodeGen/X86/ret-addr.ll

[Dale Johannesen]

This doesn't do the right thing for Depth > 1.  I don't think we need  
to short term, but the test should be ==1 and a comment should  
indicate this.

On Jan 16, 2009, at 11:25 AMPST, Bill Wendling wrote:
>
> +  if (Depth > 0) {
> +    SDValue FrameAddr = LowerFRAMEADDR(Op, DAG);
> +    SDValue Offset =
> +      DAG.getConstant(TD->getPointerSize(),
> +                      Subtarget->is64Bit() ? MVT::i64 : MVT::i32);
> +    return DAG.getLoad(getPointerTy(), DAG.getEntryNode(),
> +                       DAG.getNode(ISD::ADD, getPointerTy(),  
> FrameAddr, Offset),
> +                       NULL, 0);
> +  }