Re: [PATCH] arm: testsuite: improve guard checks for arm_neon.h

2025-03-06 Thread Christophe Lyon
On Thu, 6 Mar 2025 at 11:03, Christophe Lyon wrote: > > On Wed, 5 Mar 2025 at 12:59, Richard Earnshaw wrote: > > > > The header file arm_neon.h provides the Advanced SIMD intrinsics that > > are available on armv7 or later A & R profile cores. However, they > > are not compatible with M-profile

Re: [PATCH] arm: testsuite: improve guard checks for arm_neon.h

2025-03-06 Thread Christophe Lyon
On Wed, 5 Mar 2025 at 12:59, Richard Earnshaw wrote: > > The header file arm_neon.h provides the Advanced SIMD intrinsics that > are available on armv7 or later A & R profile cores. However, they > are not compatible with M-profile and we also need to ensure that the > FP instructions are enabled

[PATCH] arm: testsuite: improve guard checks for arm_neon.h

2025-03-05 Thread Richard Earnshaw
The header file arm_neon.h provides the Advanced SIMD intrinsics that are available on armv7 or later A & R profile cores. However, they are not compatible with M-profile and we also need to ensure that the FP instructions are enabled (with -mfloat-abi=softfp/hard). That leads to some complicated

Re: [PATCH] arm: testsuite: Adapt mve-vabs.c to improved codegen

2025-02-04 Thread Thiago Jung Bauermann
Christophe Lyon writes: > On Sun, 2 Feb 2025 at 21:18, Thiago Jung Bauermann > wrote: >> >> Since commit r15-491-gc290e6a0b7a9de this failure happens on on >> armv8l-linux-gnueabihf and arm-eabi: >> >> Running gcc:gcc.target/arm/simd/simd.exp ... >> gcc.target/arm/simd/mve-vabs.c: memmove foun

Re: [PATCH] arm: testsuite: Adapt mve-vabs.c to improved codegen

2025-02-03 Thread Christophe Lyon
On Sun, 2 Feb 2025 at 21:18, Thiago Jung Bauermann wrote: > > Since commit r15-491-gc290e6a0b7a9de this failure happens on on > armv8l-linux-gnueabihf and arm-eabi: > > Running gcc:gcc.target/arm/simd/simd.exp ... > gcc.target/arm/simd/mve-vabs.c: memmove found 0 times > FAIL: gcc.target/arm/simd/

[PATCH] arm: testsuite: Adapt mve-vabs.c to improved codegen

2025-02-02 Thread Thiago Jung Bauermann
Since commit r15-491-gc290e6a0b7a9de this failure happens on on armv8l-linux-gnueabihf and arm-eabi: Running gcc:gcc.target/arm/simd/simd.exp ... gcc.target/arm/simd/mve-vabs.c: memmove found 0 times FAIL: gcc.target/arm/simd/mve-vabs.c scan-assembler-times memmove 3 In PR PR target/116010, Andre

Re: [PATCH] arm,testsuite: Add -mtune=cortex-m55 to dlstp-int8x16.c

2024-12-16 Thread Richard Earnshaw (lists)
On 06/12/2024 16:09, Christophe Lyon wrote: > Like dlstp-compile-asm-1.c, this test would fail if GCC is configured > with non-default options, such as -mtune=cortex-a9. > > Force -mtune=cortex-m55 to avoid this unexpected issue. > > gcc/testsuite/ChangeLog: > > * gcc.target/arm/mve/dlstp-

Re: [PATCH] arm,testsuite: Add -mtune=cortex-m55 to dlstp-int8x16.c

2024-12-06 Thread Richard Earnshaw (lists)
On 06/12/2024 16:09, Christophe Lyon wrote: > Like dlstp-compile-asm-1.c, this test would fail if GCC is configured > with non-default options, such as -mtune=cortex-a9. > > Force -mtune=cortex-m55 to avoid this unexpected issue. > > gcc/testsuite/ChangeLog: > > * gcc.target/arm/mve/dlstp-

[PATCH] arm,testsuite: Add -mtune=cortex-m55 to dlstp-int8x16.c

2024-12-06 Thread Christophe Lyon
Like dlstp-compile-asm-1.c, this test would fail if GCC is configured with non-default options, such as -mtune=cortex-a9. Force -mtune=cortex-m55 to avoid this unexpected issue. gcc/testsuite/ChangeLog: * gcc.target/arm/mve/dlstp-int8x16.c: Add -mtune=cortex-m55 --- gcc/testsuite/gcc.ta

Re: [PATCH] arm, testsuite: Add -mtune=cortex-m55 to dlstp-compile-asm-1.c test.

2024-12-06 Thread Richard Earnshaw (lists)
On 06/12/2024 10:02, Christophe Lyon wrote: > This test would fail if GCC is configured with non-default options, > such as -mtune=cortex-a9. > > This 'unexpected' scheduling makes the DLSTP optimization generate > subslr, #16 > bhi .L4 > lctp > pop {r4, r5, pc}

[PATCH] arm, testsuite: Add -mtune=cortex-m55 to dlstp-compile-asm-1.c test.

2024-12-06 Thread Christophe Lyon
This test would fail if GCC is configured with non-default options, such as -mtune=cortex-a9. This 'unexpected' scheduling makes the DLSTP optimization generate subslr, #16 bhi .L4 lctp pop {r4, r5, pc} .L4: sub ip, ip, #16 b i

Re: [PATCH] arm, testsuite: Adjust Arm tests after c23 changes

2024-11-29 Thread Christophe Lyon
FTR this patch is superseded by Andre's patch: https://gcc.gnu.org/pipermail/gcc-patches/2024-November/670378.html On Thu, 28 Nov 2024 at 11:12, Christophe Lyon wrote: > > After the recent c23, GCC complains because the testcase calls f() > with a parameter whereas the prototype has none. > >

[PATCH] arm, testsuite: Adjust Arm tests after c23 changes

2024-11-28 Thread Christophe Lyon
After the recent c23, GCC complains because the testcase calls f() with a parameter whereas the prototype has none. gcc/testsuite/ChangeLog * gcc.target/arm/mve/dlstp-loop-form.c: Fix f() prototype. --- gcc/testsuite/gcc.target/arm/mve/dlstp-loop-form.c | 2 +- 1 file changed, 1 i

[PATCH] arm: testsuite: fix issues relating to fp16 alternative testing

2024-02-08 Thread Richard Earnshaw
The v*_fp16_xN_1.c tests on Arm have been unstable since they were added. This is not a problem with the tests themselves, or even the patches that were added, but with the testsuite infrastructure. It turned out that another set of dg- tests for fp16 were corrupting the cached set of options us

Re: [PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic

2023-12-06 Thread Richard Earnshaw
Sorry, I only just spotted this while looking at something else. On 23/05/2023 15:41, Christophe Lyon via Gcc-patches wrote: Glibc defines int32_t as 'int' while newlib defines it as 'long int'. Although these correspond to the same size, g++ complains when using the

Re: [PATCH] ARM/testsuite: Use non-capturing parentheses with pr53447-5.c

2023-11-22 Thread Richard Earnshaw
On 22/11/2023 01:40, Maciej W. Rozycki wrote: Use non-capturing parentheses for the subexpressions used with `scan-assembler-times', to avoid a quirk with double-counting.     gcc/testsuite/     * gcc.target/arm/pr53447-5.c: Use non-capturing parentheses with     `scan-assemble

[PATCH] ARM/testsuite: Use non-capturing parentheses with pr53447-5.c

2023-11-21 Thread Maciej W. Rozycki
Use non-capturing parentheses for the subexpressions used with `scan-assembler-times', to avoid a quirk with double-counting. gcc/testsuite/ * gcc.target/arm/pr53447-5.c: Use non-capturing parentheses with `scan-assembler-times'. --- Hi, The `scan-assembler-times' quirk

RE: [PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic

2023-05-30 Thread Kyrylo Tkachov via Gcc-patches
Ok. Thanks, Kyrill From: Christophe Lyon Sent: Tuesday, May 30, 2023 4:44 PM To: Kyrylo Tkachov Cc: gcc-patches@gcc.gnu.org; Stam Markianos-Wright Subject: Re: [PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic Ping? On Tue, 23 May 2023 at 16:59, Stamatis

Re: [PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic

2023-05-30 Thread Christophe Lyon via Gcc-patches
Ping? On Tue, 23 May 2023 at 16:59, Stamatis Markianos-Wright < stam.markianos-wri...@arm.com> wrote: > > On 23/05/2023 15:41, Christophe Lyon wrote: > > Glibc defines int32_t as 'int' while newlib defines it as 'long int'. > > > > Although these correspond to the same size, g++ complains when u

RE: [PATCH] [arm][testsuite]: Fix ACLE data-intrinsics testcases

2023-05-30 Thread Kyrylo Tkachov via Gcc-patches
> -Original Message- > From: Christophe Lyon > Sent: Tuesday, May 30, 2023 3:00 PM > To: gcc-patches@gcc.gnu.org; Kyrylo Tkachov ; > Chris Sidebottom > Cc: Christophe Lyon > Subject: [PATCH] [arm][testsuite]: Fix ACLE data-intrinsics testcases > > data-i

[PATCH] [arm][testsuite]: Fix ACLE data-intrinsics testcases

2023-05-30 Thread Christophe Lyon via Gcc-patches
data-intrinsics-assembly.c forces -march=armv6 using dg-add-options arm_arch_v6, which implicitly adds -mfloat-abi=softfp. However, for a toolchain configured for arm-linux-gnueabihf and --with-arch=armv7-a, the testcase will fail when including arm_acle.h (which includes stdint.h, which will fail

Re: [PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic

2023-05-23 Thread Stamatis Markianos-Wright via Gcc-patches
On 23/05/2023 15:41, Christophe Lyon wrote: Glibc defines int32_t as 'int' while newlib defines it as 'long int'. Although these correspond to the same size, g++ complains when using the

[PATCH] [arm] testsuite: make mve_intrinsic_type_overloads-int.c libc-agnostic

2023-05-23 Thread Christophe Lyon via Gcc-patches
Glibc defines int32_t as 'int' while newlib defines it as 'long int'. Although these correspond to the same size, g++ complains when using the

Re: [PATCH] arm/testsuite: Fix testcase for PR99977

2021-05-19 Thread Christophe Lyon via Gcc-patches
On Wed, 19 May 2021 at 16:40, Richard Earnshaw wrote: > > > > On 19/05/2021 09:10, Christophe Lyon via Gcc-patches wrote: > > Some targets (eg arm-none-uclinuxfdpiceabi) do not support Thumb-1, > > and since the testcase forces -march=armv8-m.base, we need to check > > whether this option is actua

Re: [PATCH] arm/testsuite: Fix testcase for PR99977

2021-05-19 Thread Richard Earnshaw via Gcc-patches
On 19/05/2021 09:10, Christophe Lyon via Gcc-patches wrote: Some targets (eg arm-none-uclinuxfdpiceabi) do not support Thumb-1, and since the testcase forces -march=armv8-m.base, we need to check whether this option is actually supported. Using dg-add-options arm_arch_v8m_base ensure that we

[PATCH] arm/testsuite: Fix testcase for PR99977

2021-05-19 Thread Christophe Lyon via Gcc-patches
Some targets (eg arm-none-uclinuxfdpiceabi) do not support Thumb-1, and since the testcase forces -march=armv8-m.base, we need to check whether this option is actually supported. Using dg-add-options arm_arch_v8m_base ensure that we pass -mthumb as needed too. 2021-05-19 Christophe Lyon

Re: [PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard

2020-11-27 Thread Andrea Corallo via Gcc-patches
Andrea Corallo via Gcc-patches writes: > "Richard Earnshaw (lists)" writes: > >> On 26/11/2020 13:53, Andrea Corallo via Gcc-patches wrote: >>> Hi all, >>> >>> I'd like to submit the following simple patch to clean some Low Loop >>> Overhead test failing on hard float configurations. >>> >>> l

Re: [PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard

2020-11-26 Thread Andrea Corallo via Gcc-patches
"Richard Earnshaw (lists)" writes: > On 26/11/2020 13:53, Andrea Corallo via Gcc-patches wrote: >> Hi all, >> >> I'd like to submit the following simple patch to clean some Low Loop >> Overhead test failing on hard float configurations. >> >> lob2.c and lob5.c are failing with: "'-mfloat-abi=ha

RE: [PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard

2020-11-26 Thread Kyrylo Tkachov via Gcc-patches
Hi Andrea, > -Original Message- > From: Andrea Corallo > Sent: 26 November 2020 13:54 > To: gcc-patches@gcc.gnu.org > Cc: Kyrylo Tkachov ; Richard Earnshaw > ; nd > Subject: [PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard > > Hi all, > >

Re: [PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard

2020-11-26 Thread Richard Earnshaw (lists) via Gcc-patches
On 26/11/2020 13:53, Andrea Corallo via Gcc-patches wrote: > Hi all, > > I'd like to submit the following simple patch to clean some Low Loop > Overhead test failing on hard float configurations. > > lob2.c and lob5.c are failing with: "'-mfloat-abi=hard': selected > processor lacks an FPU". >

[PATCH] arm: [testsuite] fix lob tests for -mfloat-abi=hard

2020-11-26 Thread Andrea Corallo via Gcc-patches
nd -mfloat-abi=hard may not be used together". Okay for trunk? Thanks Andrea >From ac47dfed0230d9fe596d27331b04ef194633927d Mon Sep 17 00:00:00 2001 From: Andrea Corallo Date: Thu, 26 Nov 2020 12:33:18 +0100 Subject: [PATCH] arm: [testsuite] fix lob tests for -mfloa

Re: [PATCH][ARM][testsuite] Fix address of sg stubs in CMSE tests

2019-07-09 Thread Kyrill Tkachov
Hi Christophe, On 7/2/19 3:41 PM, Christophe Lyon wrote: Hi, While running the GCC testsuite with an armv8-m target, I noticed that a few tests where causing the BFD linker to crash. I opened PR ld/24709 for this [1], but fixing it properly is tricky and not worth the headache. I "fixed" the l

Re: [PATCH][ARM][testsuite] Fix address of sg stubs in CMSE tests

2019-07-08 Thread Christophe Lyon
ping? I think that's almost obvious? And maybe should be applied to release branches. Christophe On Tue, 2 Jul 2019 at 16:41, Christophe Lyon wrote: > > Hi, > > While running the GCC testsuite with an armv8-m target, I noticed that > a few tests where causing the BFD linker to crash. I opened P

[PATCH][ARM][testsuite] Fix address of sg stubs in CMSE tests

2019-07-02 Thread Christophe Lyon
Hi, While running the GCC testsuite with an armv8-m target, I noticed that a few tests where causing the BFD linker to crash. I opened PR ld/24709 for this [1], but fixing it properly is tricky and not worth the headache. I "fixed" the linker so that it emits a useful error message instead of cr

[PATCH][arm][testsuite] Fix -march tests in effective target checks auto-generation

2018-01-17 Thread Kyrill Tkachov
Hi all, There is a typo in the armv8.1-a and armv8.2-a effective target check generators. They are not actually used anywhere in the testsuite as far as I can tell, but the fix is obvious. Committing to trunk. Thanks, Kyrill 2018-01-17 Kyrylo Tkachov * lib/target-supports.exp: Fix -ma

Re: [PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-12-05 Thread Kyrill Tkachov
On 01/12/17 15:43, Charles Baylis wrote: On 30 November 2017 at 15:56, Kyrill Tkachov wrote: So is it the case that you don't run any arm tests that include arm_neon.h in your configuration? No, it is only the case that any arm test which includes arm_neon.h (in fact, any system header) *an

Re: [PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-12-01 Thread Charles Baylis
On 30 November 2017 at 15:56, Kyrill Tkachov wrote: > > So is it the case that you don't run any arm tests that include arm_neon.h > in your configuration? No, it is only the case that any arm test which includes arm_neon.h (in fact, any system header) *and* uses dg-add-options -mfloat-abi=hard

Re: [PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-11-30 Thread Kyrill Tkachov
On 27/11/17 19:23, Charles Baylis wrote: On 27 November 2017 at 17:47, Kyrill Tkachov wrote: Hi Charles, On 27/11/17 17:03, Charles Baylis wrote: Some of the new tests in addr-modes-float.c, which were introduced for the rework of addressing modes costs [1] fail when GCC is configured to de

Re: [PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-11-27 Thread Charles Baylis
On 27 November 2017 at 17:47, Kyrill Tkachov wrote: > Hi Charles, > > On 27/11/17 17:03, Charles Baylis wrote: >> >> Some of the new tests in addr-modes-float.c, which were introduced for >> the rework of addressing modes costs [1] fail when GCC is configured >> to default to a softfp calling con

Re: [PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-11-27 Thread Kyrill Tkachov
Hi Charles, On 27/11/17 17:03, Charles Baylis wrote: Some of the new tests in addr-modes-float.c, which were introduced for the rework of addressing modes costs [1] fail when GCC is configured to default to a softfp calling convention. Fix this by annotating the test functions with __attribute__

[PATCH] ARM testsuite: force hardfp for addr-modes-float.c

2017-11-27 Thread Charles Baylis
_STORE_VEC): Likewise. (PRE_LOAD): Likewise. (POST_LOAD): Likewise. (POST_LOAD_VEC): Likewise. From c8743026e53429131e6677aaca7b0840ecc11e25 Mon Sep 17 00:00:00 2001 From: Charles Baylis Date: Fri, 24 Nov 2017 16:24:18 + Subject: [PATCH] [ARM] testsuite: force hardfp in

Re: [PATCH][ARM,testsuite] Skip copysign_softfloat_1.c on hard-float targets

2017-11-10 Thread Kyrill Tkachov
Hi Christophe, On 10/11/17 08:43, Christophe Lyon wrote: Hi, The attached testsuite patch makes gcc.target/arm/copysign_softfloat_1.c UNSUPPORTED on arm-none-linux-gnueabihf, rather than FAIL/UNRESOLVED because of a link failure since the toolchain startup code is in hard-float ABI while the te

[PATCH][ARM,testsuite] Skip copysign_softfloat_1.c on hard-float targets

2017-11-10 Thread Christophe Lyon
Hi, The attached testsuite patch makes gcc.target/arm/copysign_softfloat_1.c UNSUPPORTED on arm-none-linux-gnueabihf, rather than FAIL/UNRESOLVED because of a link failure since the toolchain startup code is in hard-float ABI while the testcase is compiled with -mfloat-abi=soft. Tested on arm* co

Re: [Patch, ARM, testsuite] Add -mfloat-abi=hard to arm_neon_ok

2017-06-26 Thread Kyrill Tkachov
Hi Christophe, On 07/06/17 10:13, Christophe Lyon wrote: Hi, On 2 June 2017 at 16:19, Christophe Lyon wrote: Hi, I have recently updated the dejagnu version I use for cross-testing arm and aarch64 toolchains to 1.6+. One of the side effects was mentioned by Jonathan in https://gcc.gnu.org/m

Re: [Patch, ARM, testsuite] Add -mfloat-abi=hard to arm_neon_ok

2017-06-26 Thread Christophe Lyon
ping? On 16 June 2017 at 17:39, Christophe Lyon wrote: > ping? > > On 7 June 2017 at 11:13, Christophe Lyon wrote: >> Hi, >> >> >> On 2 June 2017 at 16:19, Christophe Lyon wrote: >>> Hi, >>> >>> I have recently updated the dejagnu version I use for >>> cross-testing arm and aarch64 toolchains t

Re: [Patch, ARM, testsuite] Add -mfloat-abi=hard to arm_neon_ok

2017-06-16 Thread Christophe Lyon
ping? On 7 June 2017 at 11:13, Christophe Lyon wrote: > Hi, > > > On 2 June 2017 at 16:19, Christophe Lyon wrote: >> Hi, >> >> I have recently updated the dejagnu version I use for >> cross-testing arm and aarch64 toolchains to 1.6+. One of the side >> effects was mentioned by Jonathan in >> htt

Re: [Patch, ARM, testsuite] Add -mfloat-abi=hard to arm_neon_ok

2017-06-07 Thread Christophe Lyon
Hi, On 2 June 2017 at 16:19, Christophe Lyon wrote: > Hi, > > I have recently updated the dejagnu version I use for > cross-testing arm and aarch64 toolchains to 1.6+. One of the side > effects was mentioned by Jonathan in > https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01267.html. Since I > use

[Patch, ARM, testsuite] Add -mfloat-abi=hard to arm_neon_ok

2017-06-02 Thread Christophe Lyon
Hi, I have recently updated the dejagnu version I use for cross-testing arm and aarch64 toolchains to 1.6+. One of the side effects was mentioned by Jonathan in https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01267.html. Since I use multilibs to test many configurations, I noticed several changes in

Fwd: Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-12-01 Thread Thomas Preudhomme
Hi, We have decided to backport this patch fixing testing for ARMv8-M Baseline to our embedded-6-branch. *** gcc/testsuite/ChangeLog *** 2016-07-15 Thomas Preud'homme * lib/target-supports.exp (add_options_for_arm_arch_v6m): Add -mfloat-abi=soft option. (add_optio

Re: [PATCH, ARM/testsuite 6/7, ping4] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-28 Thread Mike Stump
On Nov 28, 2016, at 8:52 AM, Thomas Preudhomme wrote: > > Ping? Ok. > On 17/11/16 20:42, Thomas Preudhomme wrote: >> Ping? >> >> Best regards, >> >> Thomas >> >> On 08/11/16 13:35, Thomas Preudhomme wrote: >>> Ping, >>> >>> Best regards, >>> >>> Thomas >>> >>> On 02/11/16 10:04, Thomas P

Re: [PATCH, ARM/testsuite 6/7, ping4] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-28 Thread Thomas Preudhomme
Hi Richard, Ping? Best regards, Thomas On 17/11/16 20:42, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 08/11/16 13:35, Thomas Preudhomme wrote: Ping, Best regards, Thomas On 02/11/16 10:04, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 28/10/16 10:49, Thomas Preu

Re: [PATCH, ARM/testsuite 6/7, ping3] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-17 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 08/11/16 13:35, Thomas Preudhomme wrote: Ping, Best regards, Thomas On 02/11/16 10:04, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 28/10/16 10:49, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Th

Re: [PATCH, ARM/testsuite 6/7, ping2] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-08 Thread Thomas Preudhomme
Ping, Best regards, Thomas On 02/11/16 10:04, Thomas Preudhomme wrote: Ping? Best regards, Thomas On 28/10/16 10:49, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not ca

Re: [PATCH, ARM/testsuite 6/7, ping] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-11-02 Thread Thomas Preudhomme
Ping? Best regards, Thomas On 28/10/16 10:49, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct p

Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-28 Thread Thomas Preudhomme
On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards, Thomas On 22/09/16 14:49, Thomas Preudhomme

Re: [arm-embedded] [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-27 Thread Thomas Preudhomme
On 22/09/16 17:42, Thomas Preudhomme wrote: Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Sorry, I meant ARM/embedded-5-branch. Best regards, Thomas

Re: [PATCH, ARM/testsuite 6/7, ping] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-10-03 Thread Thomas Preudhomme
On 22/09/16 17:15, Thomas Preudhomme wrote: On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards,

[arm-embedded] [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-09-22 Thread Thomas Preudhomme
Hi, We've decided to apply the following patch to ARM/embedded-6-branch. Best regards, Thomas --- Begin Message --- Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards, Thomas On 22/09/16

Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-09-22 Thread Thomas Preudhomme
On 22/09/16 16:47, Richard Earnshaw (lists) wrote: On 22/09/16 15:51, Thomas Preudhomme wrote: Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards, Thomas On 22/09/16 14:49, Thomas Preudhomme

Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-09-22 Thread Richard Earnshaw (lists)
On 22/09/16 15:51, Thomas Preudhomme wrote: > Sorry, noticed an error in the patch. It was not caught during testing > because GCC was built with --with-mode=thumb. Correct patch attached. > > Best regards, > > Thomas > > On 22/09/16 14:49, Thomas Preudhomme wrote: >> Hi, >> >> ARMv6-M and ARMv8

Re: [PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-09-22 Thread Thomas Preudhomme
Sorry, noticed an error in the patch. It was not caught during testing because GCC was built with --with-mode=thumb. Correct patch attached. Best regards, Thomas On 22/09/16 14:49, Thomas Preudhomme wrote: Hi, ARMv6-M and ARMv8-M Baseline only support soft float ABI. Therefore, the arm_arch_

[PATCH, ARM/testsuite 6/7] Force soft float in ARMv6-M and ARMv8-M Baseline options

2016-09-22 Thread Thomas Preudhomme
Hi, ARMv6-M and ARMv8-M Baseline only support soft float ABI. Therefore, the arm_arch_v8m_base add option should pass -mfloat-abi=soft, much like -mthumb is passed for architectures that only support Thumb instruction set. This patch adds -mfloat-abi=soft to both arm_arch_v6m and arm_arch_v8m_

Re: [PATCH][ARM][Testsuite] Fix prototype in vst1Q_laneu64-1.c

2016-07-06 Thread Christophe Lyon
On 6 July 2016 at 15:04, Wilco Dijkstra wrote: > Fix prototype in vst1Q_laneu64-1.c to unsigned char* so it passes. > > Committed as trivial fix. > > ChangeLog > 2016-07-06 Wilco Dijkstra > > gcc/testsuite/ > * gcc.target/arm/vst1Q_laneu64-1.c (foo): Use unsigned char*. Thanks for

[PATCH][ARM][Testsuite] Fix prototype in vst1Q_laneu64-1.c

2016-07-06 Thread Wilco Dijkstra
Fix prototype in vst1Q_laneu64-1.c to unsigned char* so it passes. Committed as trivial fix. ChangeLog 2016-07-06 Wilco Dijkstra gcc/testsuite/ * gcc.target/arm/vst1Q_laneu64-1.c (foo): Use unsigned char*. --- diff --git a/gcc/testsuite/gcc.target/arm/vst1Q_laneu64-1.c b/gcc/tes

Re: [PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

2016-03-19 Thread Ramana Radhakrishnan
On Fri, Mar 18, 2016 at 10:31 AM, Andre Vieira (lists) wrote: > On 16/07/15 16:31, Kyrill Tkachov wrote: >> Hi all, >> >> This scan-assembler test was failing for me when testing with an >> explicit /-march=armv7-a variant because >> it clashed with the -mcpu=cortex-m7 and overrode it. >> >> This

Re: [PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

2016-03-19 Thread Andre Vieira (lists)
On 16/07/15 16:31, Kyrill Tkachov wrote: > Hi all, > > This scan-assembler test was failing for me when testing with an > explicit /-march=armv7-a variant because > it clashed with the -mcpu=cortex-m7 and overrode it. > > This patch skips the test if the user forces an incompatible -march or > -m

Re: [PATCH][ARM,testsuite] fix pragma_cpp_fma testcase

2016-03-07 Thread Kyrill Tkachov
On 06/03/16 13:58, Christophe Lyon wrote: Hi, In commit r233654, Christian introduced a new test: pragma_cpp_fma. Unfortunately, this test fails when gcc is configured --with-fpu >= neonvfpv4: __ARM_FEATURE_FMA is still defined after the last pop_options. To address this, I propose to simply

[PATCH][ARM,testsuite] fix pragma_cpp_fma testcase

2016-03-06 Thread Christophe Lyon
Hi, In commit r233654, Christian introduced a new test: pragma_cpp_fma. Unfortunately, this test fails when gcc is configured --with-fpu >= neonvfpv4: __ARM_FEATURE_FMA is still defined after the last pop_options. To address this, I propose to simply force fpu=vfp via a pragma at the beginning o

Re: FW: [PATCH, ARM/testsuite] Fix thumb2-slow-flash-data.c failures

2015-11-09 Thread Kyrill Tkachov
On 09/11/15 08:21, Thomas Preud'homme wrote: [Forwarding to gcc-patches, doh!] Best regards, Thomas [PATCH, ARM/testsuite] Fix thumb2-slow-flash-data.c failures.eml Subject: [PATCH, ARM/testsuite] Fix thumb2-slow-flash-data.c failures From: Thomas Preud'homme Date: 09/11/15

FW: [PATCH, ARM/testsuite] Fix thumb2-slow-flash-data.c failures

2015-11-09 Thread Thomas Preud'homme
[Forwarding to gcc-patches, doh!] Best regards, Thomas --- Begin Message --- Hi, ARM-specific thumb2-slow-flash-data.c testcase shows 2 failures when running for arm-none-eabi with -mcpu=cortex-m7: FAIL: gcc.target/arm/thumb2-slow-flash-data.c (test for excess errors) FAIL: gcc.target/arm/thu

Re: [PATCH][ARM][testsuite] Fix FAIL: gcc.target/arm/macro_defs0.c and macro_defs1.c when -marm forced

2015-07-21 Thread Kyrill Tkachov
On 13/07/15 17:01, Kyrill Tkachov wrote: Hi Mantas, On 05/03/15 10:14, Mantas Mikaitis wrote: Hello, Tests gcc.target/arm/macro_defs0.c and gcc.target/arm/macro_defs1.c fail in multilib which forces -marm as pointed out in this message: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00483.html

[PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

2015-07-16 Thread Kyrill Tkachov
Hi all, This scan-assembler test was failing for me when testing with an explicit /-march=armv7-a variant because it clashed with the -mcpu=cortex-m7 and overrode it. This patch skips the test if the user forces an incompatible -march or -mcpu option. The test now appears as UNSUPPORTED in the

[PATCH][ARM][testsuite][committed] Add -mfloat-abi=softfp to some xscale tests

2015-07-15 Thread Kyrill Tkachov
Hi all, This patch adds -mfloat-abi=softfp to a couple of tests that check xscale functionality and an appropriate skipping directive. This helps avoid "unimplemented Thumb-1 hard float ABI" errors when testing the arm-none-linux-gnueabihf/-mthumb variant. With this patch the two tests PASS on

Re: [PATCH][ARM][testsuite] Fix FAIL: gcc.target/arm/macro_defs0.c and macro_defs1.c when -marm forced

2015-07-13 Thread Kyrill Tkachov
Hi Mantas, On 05/03/15 10:14, Mantas Mikaitis wrote: Hello, Tests gcc.target/arm/macro_defs0.c and gcc.target/arm/macro_defs1.c fail in multilib which forces -marm as pointed out in this message: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00483.html . This patch will cause these tests to be

Ping: [PATCH][ARM][testsuite] Fix FAIL: gcc.target/arm/macro_defs0.c and macro_defs1.c when -marm forced

2015-07-10 Thread Mantas Mikaitis
Pinging this patch. Thank you, - Mantas On 05/03/15 10:14, Mantas Mikaitis wrote: Hello, Tests gcc.target/arm/macro_defs0.c and gcc.target/arm/macro_defs1.c fail in multilib which forces -marm as pointed out in this message: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00483.html . This patc

Re: [Patch][ARM,testsuite] Fix gcc.target/arm/thumb-ifcvt.c

2015-06-16 Thread Kyrill Tkachov
On 16/06/15 11:09, Christophe Lyon wrote: Hi, Since Kyrill's r224367 (Restrict MAX_CONDITIONAL_EXECUTE when -mrestrict-it is in place), gcc.target/arm/thumb-ifcvt.c fails when testing a compiler configured to generate armv8 code by default (I used --with-cpu=cortex-a57 for instance). This is b

[Patch][ARM,testsuite] Fix gcc.target/arm/thumb-ifcvt.c

2015-06-16 Thread Christophe Lyon
Hi, Since Kyrill's r224367 (Restrict MAX_CONDITIONAL_EXECUTE when -mrestrict-it is in place), gcc.target/arm/thumb-ifcvt.c fails when testing a compiler configured to generate armv8 code by default (I used --with-cpu=cortex-a57 for instance). This is because -mrestrict-it is on by default for arm

[PATCH][ARM][testsuite] Fix FAIL: gcc.target/arm/macro_defs0.c and macro_defs1.c when -marm forced

2015-03-05 Thread Mantas Mikaitis
Hello, Tests gcc.target/arm/macro_defs0.c and gcc.target/arm/macro_defs1.c fail in multilib which forces -marm as pointed out in this message: https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00483.html . This patch will cause these tests to be classified as unsupported rather than FAIL. Ok f

RE: [PATCH, ARM, testsuite] Improve scd42-1.c for UAL

2015-01-25 Thread Terry Guo
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Ramana Radhakrishnan > Sent: Friday, January 23, 2015 6:35 PM > To: Tony Liu > Cc: gcc-patches; Ramana Radhakrishnan; Richard Earnshaw > Subject: Re: [

Re: [PATCH, ARM, testsuite] Improve scd42-1.c for UAL

2015-01-23 Thread Ramana Radhakrishnan
On Thu, Jan 15, 2015 at 12:10 PM, Tony Liu wrote: > Hi, > > This is the patch to improve the test case gcc.target/arm/scd42-1.c for both > UAL and non-UAL. It now checks UAL format assembly code for Thumb1 and > Thumb2 while non-UAL format assembly code for ARM mode. OK. Ramana > > With this pa

Ping: [PATCH, ARM, testsuite] Improve scd42-1.c for UAL

2015-01-22 Thread Tony Liu
Ping -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Tony Liu Sent: Thursday, January 15, 2015 8:10 PM To: gcc-patches@gcc.gnu.org Cc: Ramana Radhakrishnan; Richard Earnshaw Subject: [PATCH, ARM, testsuite] Improve scd42-1.c for

[PATCH, ARM, testsuite] Improve scd42-1.c for UAL

2015-01-15 Thread Tony Liu
Hi, This is the patch to improve the test case gcc.target/arm/scd42-1.c for both UAL and non-UAL. It now checks UAL format assembly code for Thumb1 and Thumb2 while non-UAL format assembly code for ARM mode. With this patch, the test passes for both cases. Thanks, Tony 2015-01-15 Tony Liu

Re: [PATCH][ARM] testsuite, use arm_eabi #3

2014-11-25 Thread Andreas Tobler
On 11.11.14 00:45, Mike Stump wrote: On Nov 10, 2014, at 2:06 PM, Andreas Tobler wrote: another one. Here I'm not really sure if there are EABI variants which do _not_ support these test cases. I think the patch is fine, just watch for any follow-on comments from an eabi/arm expert. Usuall

Re: [PATCH][ARM] testsuite, use arm_eabi #3

2014-11-10 Thread Mike Stump
On Nov 10, 2014, at 2:06 PM, Andreas Tobler wrote: > another one. Here I'm not really sure if there are EABI variants which do > _not_ support these test cases. I think the patch is fine, just watch for any follow-on comments from an eabi/arm expert. Usually they are pretty responsive.

[PATCH][ARM] testsuite, use arm_eabi #3

2014-11-10 Thread Andreas Tobler
Hi all, another one. Here I'm not really sure if there are EABI variants which do _not_ support these test cases. My target succeeds with this patch. Comments? If no (comments), ok for trunk? TIA, Andreas 2014-11-10 Andreas Tobler * g++.old-deja/g++.jason/enum6.C: Use arm_eabi

Re: [PATCH][ARM] testsuite, use arm_eabi #2

2014-11-10 Thread Mike Stump
[ sorry for dup, if any ] On Nov 10, 2014, at 1:12 PM, Andreas Tobler wrote: > As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi for > all instead of listing each OS. > > Ok for trunk? Ok.

Re: [PATCH][ARM] testsuite, use arm_eabi #2

2014-11-10 Thread Mike Stump
On Nov 10, 2014, at 1:12 PM, Andreas Tobler wrote: > As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi for > all instead of listing each OS. > > Ok for trunk? Ok.

[PATCH][ARM] testsuite, use arm_eabi #2

2014-11-10 Thread Andreas Tobler
Hi all, here a second chunk which uses arm_eabi instead of arm*-*-*eabi* and arm*-*-symbianelf*. As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi for all instead of listing each OS. Ok for trunk? TIA, Andreas 2014-11-10 Andreas Tobler * gcc.target/arm/ea

Re: [PATCH][ARM] testsuite, use arm_eabi iso arm*-*-*eabi*

2014-11-10 Thread Mike Stump
On Nov 9, 2014, at 12:33 PM, Andreas Tobler wrote: > The upcoming FreeBSD ARM target does not have eabi in the target triplet. But > it is EABI based. > Ok for trunk? Ok.

[PATCH][ARM] testsuite, use arm_eabi iso arm*-*-*eabi*

2014-11-09 Thread Andreas Tobler
Hi, the subject says it already. The upcoming FreeBSD ARM target does not have eabi in the target triplet. But it is EABI based. Patching these test cases makes them work under arm*-*-freebsd*. I have not tested them under arm*-*-*eabi*, yet. If a kind sould could help me out here? Would be

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-30 Thread Marcus Shawcroft
On 30 June 2014 09:03, Ramana Radhakrishnan wrote: > + Move the tests to gcc.target/arm/ to gcc.target/aarch64 if the > AArch64 maintainers agree. For the extra AArch64 variants guard them > with #ifdef __aarch64__ #endif. Given that the intrinsics in aarch64 are a superset of those in aarch32

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-30 Thread Ramana Radhakrishnan
>> I'd rather drop the scan-assembler. I'm not convinced that the fragile >> nature of this is required. Can you add a note to the README that says >> that this is meant to be a complete execution test for the Advanced >> SIMD intrinsics and does not cover all the assembler that is > > Sure. > >> g

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 15:04, Christophe Lyon wrote: > On 27 June 2014 14:52, Ramana Radhakrishnan wrote: >> On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon >> wrote: >>> >>> diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc >>> b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unar

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 14:55, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > wrote: >> vadd tests also show how to add directives to scan the assembly >> output. >> >> diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/binary_op.inc >> b/gcc/testsuite/gcc.target

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Christophe Lyon
On 27 June 2014 14:52, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > wrote: >> >> diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc >> b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc >> new file mode 100644 >> index 000..33f

Re: [Patch ARM/testsuite 03/22] Add binary operators: vadd, vand, vbic, veor, vorn, vorr, vsub.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon wrote: > vadd tests also show how to add directives to scan the assembly > output. > > diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/binary_op.inc > b/gcc/testsuite/gcc.target/arm/neon-intrinsics/binary_op.inc > new file mode 100644 > i

Re: [Patch ARM/testsuite 02/22] Add unary operators: vabs and vneg.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon wrote: > > diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc > b/gcc/testsuite/gcc.target/arm/neon-intrinsics/unary_op.inc > new file mode 100644 > index 000..33f9b5f > --- /dev/null > +++ b/gcc/testsuite/gcc.target/arm/neo

Re: [Patch ARM/testsuite 01/22] Neon intrinsics execution tests initial framework.

2014-06-27 Thread Ramana Radhakrishnan
On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon wrote: > * documentation (README) > * dejanu driver (neon-intrinsics.exp) > * support macros (arm-neon-ref.h, compute-ref-data.h) > * Tests for 2 intrinsics: vaba, vld1 > > diff --git a/gcc/testsuite/gcc.target/arm/neon-intrinsics/README > b/gcc/te

[Patch ARM/testsuite] Adjust flags for gcc.target/vect-noalign.c

2014-06-25 Thread Ramana Radhakrishnan
The current set of flags used in this test cause issues with multilib testing where there is no neon vs neon support. Thereby adding neon specific flags through the proper interfaces. Applied to trunk after testing on arm-none-eabi with a multilib target list of {-marm/-march=armv7-a/-mfpu=vfp

Re: [Patch ARM/testsuite 00/22] Neon intrinsics executable tests

2014-06-23 Thread Christophe Lyon
On 11 June 2014 00:03, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > wrote: >> This is patch series is a more complete version of the patch I sent >> some time ago: >> https://gcc.gnu.org/ml/gcc-patches/2013-10/msg00624.html >> >> I have created a series of patc

  1   2   >