[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