Hello,
thanks for the heads-up and sorry about the trouble,
On Thu, 31 Jul 2025, Jakub Jelinek wrote:
> Perhaps at least add
> /* { dg-options "" } */
> to avoid -pedantic, or
> /* { dg-options "-msse2" } */
(choosing the latter)
> But neither of that helps with the line 12 case,
> void foo8(int i, int j) __attribute__((sseregparm, fastcall)); /* { dg-error
> "not compatible" } */
> That is simply not diagnosed.
That should have been "cdecl, fastcall", we were iterating on this patchset
a bit internally, and it seems the last revision was not tested with -m32.
I've pushed the following fixup.
Alexander
---8<---
>From 052db6bddd206a2e100787b13c6580cf3f093516 Mon Sep 17 00:00:00 2001
From: Artemiy Granat <[email protected]>
Date: Fri, 1 Aug 2025 16:19:18 +0300
Subject: [PATCH] i386: Fix incorrect attributes-error.c test
gcc/testsuite/ChangeLog:
* gcc.target/i386/attributes-error.c: Change incorrect
sseregparm,fastcall combination to cdecl,fastcall.
---
gcc/testsuite/gcc.target/i386/attributes-error.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/gcc/testsuite/gcc.target/i386/attributes-error.c
b/gcc/testsuite/gcc.target/i386/attributes-error.c
index 935ea4db0af0..5d1c77dcca61 100644
--- a/gcc/testsuite/gcc.target/i386/attributes-error.c
+++ b/gcc/testsuite/gcc.target/i386/attributes-error.c
@@ -1,3 +1,4 @@
+/* { dg-options "-msse2" } */
/* { dg-do compile } */
/* { dg-require-effective-target ia32 } */
@@ -9,7 +10,7 @@ void foo5(int i, int j) __attribute__((sseregparm,
regparm(2)));
void foo6(int i, int j) __attribute__((stdcall, fastcall)); /* { dg-error "not
compatible" } */
void foo7(int i, int j) __attribute__((regparm(2), fastcall)); /* { dg-error
"not compatible" } */
-void foo8(int i, int j) __attribute__((sseregparm, fastcall)); /* { dg-error
"not compatible" } */
+void foo8(int i, int j) __attribute__((cdecl, fastcall)); /* { dg-error "not
compatible" } */
void foo9(int i, int j) __attribute__((thiscall, fastcall)); /* { dg-error
"not compatible" } */
void foo10(int i, int j) __attribute__((sseregparm, fastcall));
