[llvm] r261032 - [WebAssembly] Implement __builtin_frame_address.

[Dan Gohman via llvm-commits]

On Tue, Feb 16, 2016 at 4:13 PM, Ahmed Bougacha <ahmed.bougacha at gmail.com>
wrote:

> On Tue, Feb 16, 2016 at 3:48 PM, Dan Gohman via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> Author: djg
>> Date: Tue Feb 16 17:48:04 2016
>> New Revision: 261032
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=261032&view=rev
>> Log:
>> [WebAssembly] Implement __builtin_frame_address.
>>
>> --- llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
>> (original)
>> +++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp Tue Feb
>> 16 17:48:04 2016
>
> @@ -579,6 +578,21 @@ SDValue WebAssemblyTargetLowering::Lower
>>    return DAG.getTargetFrameIndex(FI, Op.getValueType());
>>  }
>>
>> +SDValue WebAssemblyTargetLowering::LowerFRAMEADDR(SDValue Op,
>> +                                                  SelectionDAG &DAG)
>> const {
>> +  // Non-zero depths are not supported by WebAssembly currently. Use the
>> +  // legalizer's default expansion, which is to return 0 (what this
>> function is
>> +  // documented to do).
>> +  if (cast<ConstantSDNode>(Op.getOperand(0))->getZExtValue() > 0)
>>
>
> How about Op->getConstantOperandVal(0) ?
>

Sounds nice; updated. Thanks!

Dan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160216/ebd07cd2/attachment.html>