[PATCH][AMDGCN] Skip test gcc/testsuite/gcc.dg/asm-4.c

Andrew Stubbs ams@codesourcery.com
Thu Dec 5 10:03:00 GMT 2019


On 05/12/2019 07:05, Harwath, Frederik wrote:
> Hi,
> the inline assembly "p" modifier ("An operand that is a valid memory address is allowed",
> cf. https://gcc.gnu.org/onlinedocs/gcc/Simple-Constraints.html#Simple-Constraints)
> is not supported on AMD GCN. This causes an ICE during the compilation of gcc.dg/asm-4.c.
> We should skip the test for the amdgcn-*-* target.

For the record (should anyone else reading this wonder), it is not 
supported because the many address spaces and their differing register 
constraints make the question of what is "a valid memory address" 
somewhat nebulous.

GCN assembler should use 'RF' or 'RM', for most purposes ('RB', 'RS', 
'RL', 'RG', and 'RD' are also available).

> Can I merge the patch below into trunk?

Yes.

Andrew



More information about the Gcc-patches mailing list