[llvm] r286137 - [AArch64] Remove dead store. Found by gcc7.

[Davide Italiano via llvm-commits]

On Mon, Nov 7, 2016 at 3:34 PM,  <mcrosier at codeaurora.org> wrote:
> Thanks, David.
>

Happy to help. And, Davide =)

>
> On 2016-11-07 14:11, Davide Italiano via llvm-commits wrote:
>>
>> Author: davide
>> Date: Mon Nov  7 13:11:25 2016
>> New Revision: 286137
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=286137&view=rev
>> Log:
>> [AArch64] Remove dead store. Found by gcc7.
>>
>> Modified:
>>     llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
>>
>> Modified: llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp
>> URL:
>>
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp?rev=286137&r1=286136&r2=286137&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp (original)
>> +++ llvm/trunk/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp Mon
>> Nov  7 13:11:25 2016
>> @@ -575,15 +575,12 @@ AArch64LoadStoreOpt::mergeNarrowZeroStor
>>        MergeForward ? getLdStBaseOp(*MergeMI) : getLdStBaseOp(*I);
>>
>>    // Which register is Rt and which is Rt2 depends on the offset order.
>> -  MachineInstr *RtMI, *Rt2MI;
>> +  MachineInstr *RtMI;
>>    if (getLdStOffsetOp(*I).getImm() ==
>> -      getLdStOffsetOp(*MergeMI).getImm() + OffsetStride) {
>> +      getLdStOffsetOp(*MergeMI).getImm() + OffsetStride)
>>      RtMI = &*MergeMI;
>> -    Rt2MI = &*I;
>> -  } else {
>> +  else
>>      RtMI = &*I;
>> -    Rt2MI = &*MergeMI;
>> -  }
>>
>>    int OffsetImm = getLdStOffsetOp(*RtMI).getImm();
>>    // Change the scaled offset from small to large type.
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits



-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare