* [PATCH] testsuite, arm: Fix up pr112337.c test
@ 2023-12-01 11:28 Saurabh Jha
2023-12-01 12:44 ` Richard Earnshaw (lists)
0 siblings, 1 reply; 6+ messages in thread
From: Saurabh Jha @ 2023-12-01 11:28 UTC (permalink / raw)
To: gcc-patches, Kyrylo Tkachov
[-- Attachment #1: Type: text/plain, Size: 847 bytes --]
Hey,
I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
"""
Tests that now work, but didn't before (2 tests):
arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
"""
Ok for trunk? I don't have commit access so could someone please commit on my behalf?
Regards,
Saurabh
gcc/testsuite/ChangeLog:
* gcc.target/arm/mve/pr112337.c: Fix the testcase
[-- Attachment #2: diff --]
[-- Type: text/plain, Size: 816 bytes --]
From 2365aae84de030bbb006edac18c9314812fc657b Mon Sep 17 00:00:00 2001
From: Saurabh Jha <saujha01@e130340.arm.com>
Date: Tue, 28 Nov 2023 13:05:58 +0000
Subject: [PATCH] testsuite: Fix up pr112337.c test
---
gcc/testsuite/gcc.target/arm/mve/pr112337.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/testsuite/gcc.target/arm/mve/pr112337.c b/gcc/testsuite/gcc.target/arm/mve/pr112337.c
index 8f491990088..d1a075ecd0e 100644
--- a/gcc/testsuite/gcc.target/arm/mve/pr112337.c
+++ b/gcc/testsuite/gcc.target/arm/mve/pr112337.c
@@ -5,8 +5,8 @@
#include <arm_mve.h>
void g(int32x4_t);
-void f(int, int, int, short, int *p) {
- int *bias = p;
+void f(int, int, int, short, int32_t *p) {
+ int32_t *bias = p;
for (;;) {
int32x4_t d = vldrwq_s32 (p);
bias += 4;
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] testsuite, arm: Fix up pr112337.c test
2023-12-01 11:28 [PATCH] testsuite, arm: Fix up pr112337.c test Saurabh Jha
@ 2023-12-01 12:44 ` Richard Earnshaw (lists)
2023-12-01 13:45 ` Christophe Lyon
0 siblings, 1 reply; 6+ messages in thread
From: Richard Earnshaw (lists) @ 2023-12-01 12:44 UTC (permalink / raw)
To: Saurabh Jha, gcc-patches, Kyrylo Tkachov, christophe.lyon
On 01/12/2023 11:28, Saurabh Jha wrote:
> Hey,
>
> I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
>
> Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
>
> """
> Tests that now work, but didn't before (2 tests):
>
> arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
> arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
> """
>
> Ok for trunk? I don't have commit access so could someone please commit on my behalf?
>
> Regards,
> Saurabh
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/arm/mve/pr112337.c: Fix the testcase
Hmm, could this be related to the changes Christophe made recently to change the way MVE vector types were set up internally? If so, this might indicate an issue that's going to affect real users with existing code.
Christophe?
R.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] testsuite, arm: Fix up pr112337.c test
2023-12-01 12:44 ` Richard Earnshaw (lists)
@ 2023-12-01 13:45 ` Christophe Lyon
2023-12-01 14:10 ` Richard Earnshaw (lists)
0 siblings, 1 reply; 6+ messages in thread
From: Christophe Lyon @ 2023-12-01 13:45 UTC (permalink / raw)
To: Richard Earnshaw (lists); +Cc: Saurabh Jha, gcc-patches, Kyrylo Tkachov
On Fri, 1 Dec 2023 at 13:44, Richard Earnshaw (lists)
<Richard.Earnshaw@arm.com> wrote:
>
> On 01/12/2023 11:28, Saurabh Jha wrote:
> > Hey,
> >
> > I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
> >
> > Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
> >
> > """
> > Tests that now work, but didn't before (2 tests):
> >
> > arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
> > arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
> > """
> >
> > Ok for trunk? I don't have commit access so could someone please commit on my behalf?
> >
> > Regards,
> > Saurabh
> >
> > gcc/testsuite/ChangeLog:
> >
> > * gcc.target/arm/mve/pr112337.c: Fix the testcase
>
>
> Hmm, could this be related to the changes Christophe made recently to change the way MVE vector types were set up internally? If so, this might indicate an issue that's going to affect real users with existing code.
>
My change was only about vector types, here the problem is with a
pointer to a scalar.
Anyway, I ran the test with my commit reverted and it still fails in
the same way, so I think this patch is needed.
Thanks,
Christophe
> Christophe?
>
> R.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] testsuite, arm: Fix up pr112337.c test
2023-12-01 13:45 ` Christophe Lyon
@ 2023-12-01 14:10 ` Richard Earnshaw (lists)
2023-12-01 16:46 ` [PATCH v2] " Saurabh Jha
0 siblings, 1 reply; 6+ messages in thread
From: Richard Earnshaw (lists) @ 2023-12-01 14:10 UTC (permalink / raw)
To: Christophe Lyon; +Cc: Saurabh Jha, gcc-patches, Kyrylo Tkachov
On 01/12/2023 13:45, Christophe Lyon wrote:
> On Fri, 1 Dec 2023 at 13:44, Richard Earnshaw (lists)
> <Richard.Earnshaw@arm.com> wrote:
>>
>> On 01/12/2023 11:28, Saurabh Jha wrote:
>>> Hey,
>>>
>>> I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
>>>
>>> Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
>>>
>>> """
>>> Tests that now work, but didn't before (2 tests):
>>>
>>> arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>> arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>> """
>>>
>>> Ok for trunk? I don't have commit access so could someone please commit on my behalf?
>>>
>>> Regards,
>>> Saurabh
>>>
>>> gcc/testsuite/ChangeLog:
>>>
>>> * gcc.target/arm/mve/pr112337.c: Fix the testcase
>>
>>
>> Hmm, could this be related to the changes Christophe made recently to change the way MVE vector types were set up internally? If so, this might indicate an issue that's going to affect real users with existing code.
>>
>
> My change was only about vector types, here the problem is with a
> pointer to a scalar.
> Anyway, I ran the test with my commit reverted and it still fails in
> the same way, so I think this patch is needed.
>
> Thanks,
>
> Christophe
>
>> Christophe?
>>
>> R.
Ok, thanks for checking. In that case, Saurabh, your patch is OK, but please change 'Fix testcase' to 'Use int32_t instead of int.'
Note that ChangeLog entries end with a full stop.
R.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] testsuite, arm: Fix up pr112337.c test
2023-12-01 14:10 ` Richard Earnshaw (lists)
@ 2023-12-01 16:46 ` Saurabh Jha
2023-12-03 16:16 ` Richard Sandiford
0 siblings, 1 reply; 6+ messages in thread
From: Saurabh Jha @ 2023-12-01 16:46 UTC (permalink / raw)
To: Richard Earnshaw (lists), Christophe Lyon; +Cc: gcc-patches, Kyrylo Tkachov
[-- Attachment #1: Type: text/plain, Size: 2050 bytes --]
On 12/1/2023 2:10 PM, Richard Earnshaw (lists) wrote:
> On 01/12/2023 13:45, Christophe Lyon wrote:
>> On Fri, 1 Dec 2023 at 13:44, Richard Earnshaw (lists)
>> <Richard.Earnshaw@arm.com> wrote:
>>> On 01/12/2023 11:28, Saurabh Jha wrote:
>>>> Hey,
>>>>
>>>> I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
>>>>
>>>> Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
>>>>
>>>> """
>>>> Tests that now work, but didn't before (2 tests):
>>>>
>>>> arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>>> arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>>> """
>>>>
>>>> Ok for trunk? I don't have commit access so could someone please commit on my behalf?
>>>>
>>>> Regards,
>>>> Saurabh
>>>>
>>>> gcc/testsuite/ChangeLog:
>>>>
>>>> * gcc.target/arm/mve/pr112337.c: Fix the testcase
>>>
>>> Hmm, could this be related to the changes Christophe made recently to change the way MVE vector types were set up internally? If so, this might indicate an issue that's going to affect real users with existing code.
>>>
>> My change was only about vector types, here the problem is with a
>> pointer to a scalar.
>> Anyway, I ran the test with my commit reverted and it still fails in
>> the same way, so I think this patch is needed.
>>
>> Thanks,
>>
>> Christophe
>>
>>> Christophe?
>>>
>>> R.
> Ok, thanks for checking. In that case, Saurabh, your patch is OK, but please change 'Fix testcase' to 'Use int32_t instead of int.'
>
> Note that ChangeLog entries end with a full stop.
>
> R.
Thank you for the feedback. Please find the updated ChangeLog below.
gcc/testsuite/ChangeLog:
* gcc.target/arm/mve/pr112337.c: Use int32_t instead of int.
[-- Attachment #2: diff --]
[-- Type: text/plain, Size: 816 bytes --]
From 2365aae84de030bbb006edac18c9314812fc657b Mon Sep 17 00:00:00 2001
From: Saurabh Jha <saujha01@e130340.arm.com>
Date: Tue, 28 Nov 2023 13:05:58 +0000
Subject: [PATCH] testsuite: Fix up pr112337.c test
---
gcc/testsuite/gcc.target/arm/mve/pr112337.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/testsuite/gcc.target/arm/mve/pr112337.c b/gcc/testsuite/gcc.target/arm/mve/pr112337.c
index 8f491990088..d1a075ecd0e 100644
--- a/gcc/testsuite/gcc.target/arm/mve/pr112337.c
+++ b/gcc/testsuite/gcc.target/arm/mve/pr112337.c
@@ -5,8 +5,8 @@
#include <arm_mve.h>
void g(int32x4_t);
-void f(int, int, int, short, int *p) {
- int *bias = p;
+void f(int, int, int, short, int32_t *p) {
+ int32_t *bias = p;
for (;;) {
int32x4_t d = vldrwq_s32 (p);
bias += 4;
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] testsuite, arm: Fix up pr112337.c test
2023-12-01 16:46 ` [PATCH v2] " Saurabh Jha
@ 2023-12-03 16:16 ` Richard Sandiford
0 siblings, 0 replies; 6+ messages in thread
From: Richard Sandiford @ 2023-12-03 16:16 UTC (permalink / raw)
To: Saurabh Jha
Cc: Richard Earnshaw (lists), Christophe Lyon, gcc-patches, Kyrylo Tkachov
Saurabh Jha <saurabh.jha@arm.com> writes:
> On 12/1/2023 2:10 PM, Richard Earnshaw (lists) wrote:
>> On 01/12/2023 13:45, Christophe Lyon wrote:
>>> On Fri, 1 Dec 2023 at 13:44, Richard Earnshaw (lists)
>>> <Richard.Earnshaw@arm.com> wrote:
>>>> On 01/12/2023 11:28, Saurabh Jha wrote:
>>>>> Hey,
>>>>>
>>>>> I introduced this test "gcc/testsuite/gcc.target/arm/mve/pr112337.c" in this commit 2365aae84de030bbb006edac18c9314812fc657b before. This had an error which I unfortunately missed. This patch fixes that test.
>>>>>
>>>>> Did regression testing on arm-none-eabi and found no regressions. Output of running gcc/contrib/compare_tests is this:
>>>>>
>>>>> """
>>>>> Tests that now work, but didn't before (2 tests):
>>>>>
>>>>> arm-eabi-aem/-marm/-march=armv7-a/-mfpu=vfpv3-d16/-mfloat-abi=softfp: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>>>> arm-eabi-aem/-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard: gcc.target/arm/mve/pr112337.c (test for excess errors)
>>>>> """
>>>>>
>>>>> Ok for trunk? I don't have commit access so could someone please commit on my behalf?
>>>>>
>>>>> Regards,
>>>>> Saurabh
>>>>>
>>>>> gcc/testsuite/ChangeLog:
>>>>>
>>>>> * gcc.target/arm/mve/pr112337.c: Fix the testcase
>>>>
>>>> Hmm, could this be related to the changes Christophe made recently to change the way MVE vector types were set up internally? If so, this might indicate an issue that's going to affect real users with existing code.
>>>>
>>> My change was only about vector types, here the problem is with a
>>> pointer to a scalar.
>>> Anyway, I ran the test with my commit reverted and it still fails in
>>> the same way, so I think this patch is needed.
>>>
>>> Thanks,
>>>
>>> Christophe
>>>
>>>> Christophe?
>>>>
>>>> R.
>> Ok, thanks for checking. In that case, Saurabh, your patch is OK, but please change 'Fix testcase' to 'Use int32_t instead of int.'
>>
>> Note that ChangeLog entries end with a full stop.
>>
>> R.
>
> Thank you for the feedback. Please find the updated ChangeLog below.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/arm/mve/pr112337.c: Use int32_t instead of int.
Thanks, pushed to trunk.
Richard
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-12-03 16:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-01 11:28 [PATCH] testsuite, arm: Fix up pr112337.c test Saurabh Jha
2023-12-01 12:44 ` Richard Earnshaw (lists)
2023-12-01 13:45 ` Christophe Lyon
2023-12-01 14:10 ` Richard Earnshaw (lists)
2023-12-01 16:46 ` [PATCH v2] " Saurabh Jha
2023-12-03 16:16 ` Richard Sandiford
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).