[llvm-commits] Patch to add support for WidenVecRes_MERGE_VALUES

Eli Friedman eli.friedman at gmail.com
Tue Aug 30 12:55:43 PDT 2011


On Tue, Aug 30, 2011 at 11:35 AM, Tobias Grosser <tobias at grosser.es> wrote:
> On 08/30/2011 01:04 PM, Villmow, Micah wrote:
>>
>> Thanks for the feedback, new revision attached.
>
> There seems one unneeded white space change. (Can be fixed on commit)
>
>> SDValue MakeLibCall(RTLIB::Libcall LC, EVT RetVT,
>>                    const SDValue *Ops, unsigned NumOps, bool isSigned,
>>                    DebugLoc dl);
>> +
>>        std::pair<SDValue, SDValue>   ExpandChainLibCall(RTLIB::Libcall
>
> Otherwise, it looks good. Let's see if Eli has any additional comments.

Looks fine.

Note that it would be nice to make sure we implement MERGE_VALUES for
all the other cases in type legalization... (integer promotion, vector
scalarization, etc.)

-Eli




More information about the llvm-commits mailing list