[llvm] r252748 - Remove unused variable in disable-assert builds. NFC.

Diego Novillo via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 08:42:58 PST 2015


On Wed, Nov 11, 2015 at 11:34 AM, David Blaikie <dblaikie at gmail.com> wrote:

>
>
> On Wed, Nov 11, 2015 at 8:14 AM, Diego Novillo via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
>> Author: dnovillo
>> Date: Wed Nov 11 10:14:52 2015
>> New Revision: 252748
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=252748&view=rev
>> Log:
>> Remove unused variable in disable-assert builds.  NFC.
>>
>> Modified:
>>     llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
>>
>> Modified: llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp?rev=252748&r1=252747&r2=252748&view=diff
>>
>> ==============================================================================
>> --- llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp (original)
>> +++ llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp Wed Nov 11 10:14:52 2015
>> @@ -9141,8 +9141,7 @@ static SDValue PerformBFICombine(SDNode
>>      SDValue From1 = ParseBFI(N, ToMask1, FromMask1);
>>
>>      APInt ToMask2, FromMask2;
>> -    SDValue From2 = ParseBFI(CombineBFI.getNode(), ToMask2, FromMask2);
>> -    assert(From1 == From2);
>> +    assert(From1 == ParseBFI(CombineBFI.getNode(), ToMask2, FromMask2));
>>
>
> This looks like the wrong transformation - presumably ParseBFI has side
> effects, like parsing the result into ToMask2 and FromMask2 - putting it
> inside the assert means those side effects will not be performed in a
> -Asserts build.
>
> The right thing in this instance is probably to add a (void)From2;
> statement either before or after the assert (we're not terribly consistent
> about whether it goes before or after)
>

Thanks, Dave.  I wasn't paying attention.  Fixed at r252751.


Diego.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151111/f3f399d7/attachment.html>


More information about the llvm-commits mailing list