[llvm] r180014 - Remove variable store that is never read.

Eric Christopher echristo at gmail.com
Mon Apr 22 07:13:17 PDT 2013


On Mon, Apr 22, 2013 at 7:02 AM, Benjamin Kramer <benny.kra at gmail.com> wrote:
>
> On 22.04.2013, at 15:51, Eric Christopher <echristo at gmail.com> wrote:
>
>> Author: echristo
>> Date: Mon Apr 22 08:51:44 2013
>> New Revision: 180014
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=180014&view=rev
>> Log:
>> Remove variable store that is never read.
>>
>> Modified:
>>    llvm/trunk/lib/Target/ARM/ARMFastISel.cpp
>>
>> Modified: llvm/trunk/lib/Target/ARM/ARMFastISel.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMFastISel.cpp?rev=180014&r1=180013&r2=180014&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/ARM/ARMFastISel.cpp (original)
>> +++ llvm/trunk/lib/Target/ARM/ARMFastISel.cpp Mon Apr 22 08:51:44 2013
>> @@ -2605,7 +2605,7 @@ unsigned ARMFastISel::ARMEmitIntExt(MVT
>>
>>   unsigned Opc;
>>   bool isBoolZext = false;
>> -  const TargetRegisterClass *RC = TLI.getRegClassFor(MVT::i32);
>> +  const TargetRegisterClass *RC = 0;
>
> If it is never read, why store a 0 to it?
>

Some reason I thought I saw a warning there when it was building...
I've fixed it. :)

-eric



More information about the llvm-commits mailing list