[Diffusion] rL259840: Fix for PR 26356

Thu Feb 4 16:59:57 PST 2016

This conflicted with Eric's r259178, so I pulled that in too. (+Eric fyi)

Merged in r259858.

Cheers,
Hans

On Thu, Feb 4, 2016 at 3:54 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> ----- Original Message -----
>> From: "Nemanja Ivanovic" <nemanja.i.ibm at gmail.com>
>> To: "Hans Wennborg" <hans at chromium.org>, "Hal Finkel" <hfinkel at anl.gov>
>> Sent: Thursday, February 4, 2016 5:22:16 PM
>> Subject: Fwd: [Diffusion] rL259840: Fix for PR 26356
>>
>> Hi Hans,
>> this should be the last of those fixes for PPC Fast Isel.
>
> Approved.
>
>  -Hal
>
>> Would you
>> be able to merge this into 3.8?
>>
>> Thank you,
>> Nemanja
>>
>>
>> ---------- Forwarded message ----------
>> From: Nemanja Ivanovic < nemanja.i.ibm at gmail.com >
>> Date: Fri, Feb 5, 2016 at 12:19 AM
>> Subject: Re: [Diffusion] rL259840: Fix for PR 26356
>> To: nemanja.i.ibm at gmail.com
>>
>>
>> nemanjai committed rL259840: Fix for PR 26356.
>>
>> Fix for PR 26356
>>
>> Using the load immediate only when the immediate (whether signed or
>> unsigned)
>> can fit in a 16-bit signed field. Namely, from -32768 to 32767 for
>> signed and
>> 0 to 65535 for unsigned. This patch also ensures that we sign-extend
>> under the
>> right conditions.
>>
>>
>> Files:
>> /llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp
>> /llvm/trunk/test/CodeGen/PowerPC/pr26356.ll
>>
>> Users:
>> nemanjai (Author)
>>
>> http://reviews.llvm.org/rL259840
>>
>>
>>
>>
>>
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory