[llvm] r259645 - Fix for PR 26381

[Hans Wennborg via llvm-commits]

Merged in r259698. Please let me know when the test lands, and I'll
merge that too.

On Wed, Feb 3, 2016 at 5:21 AM, Hal Finkel <hfinkel at anl.gov> wrote:
> Hi Hans,
>
> Please pull this into the release branch.
>
> Nemanja, please add a test case. Then also send a note to Hans so he can pull that into the release branch too.
>
>  -Hal
>
> ----- Original Message -----
>> From: "Nemanja Ivanovic via llvm-commits" <llvm-commits at lists.llvm.org>
>> To: llvm-commits at lists.llvm.org
>> Sent: Wednesday, February 3, 2016 6:53:39 AM
>> Subject: [llvm] r259645 - Fix for PR 26381
>>
>> Author: nemanjai
>> Date: Wed Feb  3 06:53:38 2016
>> New Revision: 259645
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=259645&view=rev
>> Log:
>> Fix for PR 26381
>>
>> Simple fix - Constant values were not being sign extended in
>> FastIsel.
>>
>> Modified:
>>     llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp
>>
>> Modified: llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp?rev=259645&r1=259644&r2=259645&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp (original)
>> +++ llvm/trunk/lib/Target/PowerPC/PPCFastISel.cpp Wed Feb  3 06:53:38
>> 2016
>> @@ -2105,7 +2105,7 @@ unsigned PPCFastISel::PPCMaterializeInt(
>>    }
>>
>>    // Construct the constant piecewise.
>> -  int64_t Imm = CI->getZExtValue();
>> +  int64_t Imm = UseSExt ? CI->getSExtValue() : CI->getZExtValue();
>>    if (VT == MVT::i64)
>>      return PPCMaterialize64BitInt(Imm, RC);
>>    else if (VT == MVT::i32)
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory