diff mbox series

[2/3] arm, mve: Pass -std=c99 to dlstp-loop-form.c to avoid new warning

Message ID 20241129103012.3477414-3-andre.simoesdiasvieira@arm.com
State New
Headers show
Series arm, mve: Fix DLSTP testism and issue after changes in codegen | expand

Commit Message

Andre Vieira (lists) Nov. 29, 2024, 10:30 a.m. UTC 
This fixes a testism introduced by the warning produced with the -std=c23
default.  The testcase is a reduced piece of code meant to trigger an ICE, so
there's little value in trying to change the code itself.

gcc/testsuite/ChangeLog:

	* gcc.target/arm/mve/dlstp-loop-form.c: Add -std=c99 to avoid warning
	message.
---
 gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Christophe Lyon Nov. 29, 2024, 10:59 a.m. UTC | #1 
On 11/29/24 11:30, Andre Vieira wrote:
> 
> This fixes a testism introduced by the warning produced with the -std=c23
> default.  The testcase is a reduced piece of code meant to trigger an ICE, so
> there's little value in trying to change the code itself.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/arm/mve/dlstp-loop-form.c: Add -std=c99 to avoid warning
> 	message.

Thanks indeed that's better than my proposal:
https://gcc.gnu.org/pipermail/gcc-patches/2024-November/670267.html

> ---
>   gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c b/gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c
index 08811cef568..0f9589d7756 100644
--- a/gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c
+++ b/gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c
@@ -1,6 +1,6 @@ 
 /* { dg-do compile } */
 /* { dg-require-effective-target arm_v8_1m_mve_fp_ok } */
-/* { dg-options "-Ofast" } */
+/* { dg-options "-Ofast -std=c99" } */
 /* { dg-add-options arm_v8_1m_mve_fp } */
 #pragma GCC arm "arm_mve_types.h"
 #pragma GCC arm "arm_mve.h" false