On 19/11/2024 10:24, Torbjörn SVENSSON wrote:
> Update test case to use -mcpu=unset/-march=unset feature introduced in
> r15-3606-g7d6c6a0d15c.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/arm/lto/pr96939_0.c: Use effective-target
> arm_arch_v8a.
> * gcc.target/arm/lto/pr96939_1.c: Remove dg-options.
>
> Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
> ---
> gcc/testsuite/gcc.target/arm/lto/pr96939_0.c | 4 ++--
> gcc/testsuite/gcc.target/arm/lto/pr96939_1.c | 1 -
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/arm/lto/pr96939_0.c
> b/gcc/testsuite/gcc.target/arm/lto/pr96939_0.c
> index 241ffd5da0a..3bb74bd1a1d 100644
> --- a/gcc/testsuite/gcc.target/arm/lto/pr96939_0.c
> +++ b/gcc/testsuite/gcc.target/arm/lto/pr96939_0.c
> @@ -1,7 +1,7 @@
> /* PR target/96939 */
> /* { dg-lto-do link } */
> -/* { dg-require-effective-target arm_arch_v8a_ok } */
> -/* { dg-lto-options { { -flto -O2 } } } */
> +/* { dg-require-effective-target arm_arch_v8a_link } */
> +/* { dg-lto-options { { -flto -O2 -mcpu=unset -march=armv8-a+simd+crc } } }
> */
>
> extern unsigned crc (unsigned, const void *);
> typedef unsigned (*fnptr) (unsigned, const void *);
> diff --git a/gcc/testsuite/gcc.target/arm/lto/pr96939_1.c
> b/gcc/testsuite/gcc.target/arm/lto/pr96939_1.c
> index 4afdbdaf5ad..c641b5580ab 100644
> --- a/gcc/testsuite/gcc.target/arm/lto/pr96939_1.c
> +++ b/gcc/testsuite/gcc.target/arm/lto/pr96939_1.c
> @@ -1,5 +1,4 @@
> /* PR target/96939 */
> -/* { dg-options "-march=armv8-a+simd+crc" } */
>
> #include <arm_acle.h>
>
I'm not sure this is right. The PR talks about handling streaming in of
objects built with different options, which are supposed to be recorded in the
streaming data. But your change alters what will be recorded AFAICT.
R.
