procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

testsuite: Fix toplevel-asm-1.c failure for riscv

Browse source 
On Wed, Dec 18, 2024 at 01:19:43PM +0100, Andreas Schwab wrote:
> On Dez 12 2024, Jakub Jelinek wrote:
>
> > The intent was to test %cN because %N doesn't DTRT on various targets.
> > I have a patch to add %ccN support which should then work even on riscv
> > hopefully, but unfortunately it hasn't been fully reviewed yet.
>
> That didn't change toplevel-asm-1, so the failure remains.

Yes, I've only committed what was approved.

The following patch ought to fix this (and if there are other targets which
don't really support %cN for SYMBOL_REFs even with -fno-pic, they can be
added there too; I think it is useful to test %cN on the targets where it
works though).

2024-12-19  Jakub Jelinek  <jakub@redhat.com>

	* c-c++-common/toplevel-asm-1.c: Use %cc3 %cc4 instead of %c3 %c4
	on riscv.
This commit is contained in:
Jakub Jelinek 2024-12-19 11:36:29 +01:00 • committed by Jakub Jelinek
parent d063549217
commit b117201385
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gcc/testsuite/c-c++-common/toplevel-asm-1.c
View file

@ -8,7 +8,12 @@ enum E { E0, E1 = sizeof (struct S) + 15 };
int v[42];
void foo (void) {}
/* Not all targets can use %cN even in non-pic code. */
#if defined(__riscv)
asm ("# %0 %1 %2 %cc3 %cc4 %5 %% %="
#else
asm ("# %0 %1 %2 %c3 %c4 %5 %% %="
#endif
:: "i" (sizeof (struct S)),
"i" (__builtin_offsetof (struct S, c)),
"i" (E1),