[PATCH, PING] Fix common-code ppcf128 component access on little-endian

[Alp Toker]

Nice work Ulrich, thanks! 


On 3 July 2014 18:16:00 EEST, Ulrich Weigand <Ulrich.Weigand at de.ibm.com> wrote:
>Hal Finkel <hfinkel at anl.gov> wrote on 03.07.2014 06:03:57:
>
>> > Here's an updated version of the patch that uses a helper routine
>> > hasBigEndianPartOrdering.  It also adds tests to cover the other
>> > changes.
>> >
>> > In doing so I noticed that I missed a case in ExpandRes_BITCAST,
>> > and once I added that, it turns out that the original change to
>> > getCopyToParts is not necessary. (This is because that function
>> > first uses a BITCAST to convert to an integer type, and then
>> > operates on the integer type.  The endian swap for ppc_fp128
>> > takes place during the BITCAST already; no further swap is
>> > required.)
>> >
>> > Does this look OK?
>>
>> Much better, thanks! LGTM.
>>
>> (Alp, thanks for the suggestion, the logic is indeed much clearer
>this
>way).
>
>Committed as revision 212274.
>
>Thanks again,
>Ulrich
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140703/d14e6e16/attachment.html>