[llvm] r173755 - Teach SDISel to combine fsin / fcos into a fsincos node if the following

Evan Cheng evan.cheng at apple.com
Tue Jan 29 08:43:47 PST 2013


Oops. :) I'll fix. 

Evan

Sent from my iPad

On Jan 29, 2013, at 8:08 AM, Eric Christopher <echristo at gmail.com> wrote:

> Silly nitpicking:
> 
>> +#if 1
>> +  return CallResult.first;
>> +#else
>> +  SDValue RetSin = DAG.getNode(ISD::EXTRACT_ELEMENT, dl, ArgVT,
>> +                               CallResult.first, DAG.getIntPtrConstant(0));
>> +  SDValue RetCos = DAG.getNode(ISD::EXTRACT_ELEMENT, dl, ArgVT,
>> +                               CallResult.first, DAG.getIntPtrConstant(1));
>> +
>> +  SDVTList Tys = DAG.getVTList(ArgVT, ArgVT);
>> +  return DAG.getNode(ISD::MERGE_VALUES, dl, Tys, RetSin, RetCos);
>> +#endif
> 
> #if 1? :)
> 
> -eric 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130129/ec0aa8db/attachment.html>


More information about the llvm-commits mailing list