[PATCH 1/2] c-family: Copy DECL_USER_ALIGN even if DECL_ALIGN is similar.

Jason Merrill jason@redhat.com
Tue Jun 1 19:11:27 GMT 2021


On 6/1/21 9:20 AM, Robin Dapp wrote:
>> As you say, the logic is convoluted.  Let's simplify it rather than make
>> it more convoluted.  One possibility would be to change || to | to avoid
>> the shortcut, and then
>>
>> bool note = lastalign > curalign;
>> if (note)
>>     curalign = lastalign;
> 
> I went with your suggestion in the attached v2.  Regtested and 
> bootstrapped on s390x, x86 and ppc64le.

OK.

Jason



More information about the Gcc-patches mailing list