[PATCH, i386]: Fix gcc.c-torture/compile/pr41634.c FAIL

Uros Bizjak ubizjak@gmail.com
Tue Nov 10 18:14:00 GMT 2015


On Tue, Nov 10, 2015 at 7:00 PM, Richard Henderson <rth@redhat.com> wrote:
> On 11/10/2015 06:54 PM, Uros Bizjak wrote:
>>
>> -      return "movabs{<imodesuffix>}\t{%1, %0|%0, %1}";
>> +      return "movabs{<imodesuffix>}\t{%1, %P0|[%P0], %1}";
>
>
> The thing that's missing from this, that's present in the patch that I sent
> you off-list, is the <iptrsize> thing for Intel syntax.
>
> Would you prefer to just add that back here via <iptrsize>, rather than
> using a new %v specifier like in my patch?

I have opted for the same assembly code as it was generated
previously. But, since we have macroized pattern and already available
mode attribute, I'd prefer to use <iptrsize> PTR [...]. There are
already a couple of examples using this approach in i386.md.

BTW: gas is able to determine pointer size from register name, so
having PTR prefix does not change generated object code.

Uros.



More information about the Gcc-patches mailing list