public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH v1] Test: Move target independent test cases to gcc.dg/torture
@ 2024-06-11  6:19 pan2.li
  2024-06-11 13:40 ` Jeff Law
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: pan2.li @ 2024-06-11  6:19 UTC (permalink / raw)
  To: gcc-patches
  Cc: juzhe.zhong, kito.cheng, richard.guenther, jeffreyalaw, Pan Li

From: Pan Li <pan2.li@intel.com>

The test cases of pr115387 are target independent,  at least x86
and riscv are able to reproduce.  Thus,  move these cases to
the gcc.dg/torture.

The below test suites are passed.
1. The rv64gcv fully regression test.
2. The x86 fully regression test.

gcc/testsuite/ChangeLog:

	* gcc.target/riscv/pr115387-1.c: Move to...
	* gcc.dg/torture/pr115387-1.c: ...here.
	* gcc.target/riscv/pr115387-2.c: Move to...
	* gcc.dg/torture/pr115387-2.c: ...here.

Signed-off-by: Pan Li <pan2.li@intel.com>
---
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c | 2 +-
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c (92%)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c (84%)

diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-1.c b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
similarity index 92%
rename from gcc/testsuite/gcc.target/riscv/pr115387-1.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-1.c
index a1c926977c4..fde79f66757 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-1.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
@@ -1,6 +1,6 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
+/* { dg-options "-O3" } */
 
 #define PRINTF_CHK 0x34
 
diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-2.c b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
similarity index 84%
rename from gcc/testsuite/gcc.target/riscv/pr115387-2.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-2.c
index 7183bf18dfd..0cb4b48d27b 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-2.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
@@ -1,6 +1,6 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
+/* { dg-options "-O3" } */
 
 #include <stddef.h>
 #include <stdint-gcc.h>
-- 
2.34.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1] Test: Move target independent test cases to gcc.dg/torture
  2024-06-11  6:19 [PATCH v1] Test: Move target independent test cases to gcc.dg/torture pan2.li
@ 2024-06-11 13:40 ` Jeff Law
  2024-06-11 13:45 ` Andrew Pinski
  2024-06-11 14:53 ` [PATCH v2] " pan2.li
  2 siblings, 0 replies; 7+ messages in thread
From: Jeff Law @ 2024-06-11 13:40 UTC (permalink / raw)
  To: pan2.li, gcc-patches; +Cc: juzhe.zhong, kito.cheng, richard.guenther



On 6/11/24 12:19 AM, pan2.li@intel.com wrote:
> From: Pan Li <pan2.li@intel.com>
> 
> The test cases of pr115387 are target independent,  at least x86
> and riscv are able to reproduce.  Thus,  move these cases to
> the gcc.dg/torture.
> 
> The below test suites are passed.
> 1. The rv64gcv fully regression test.
> 2. The x86 fully regression test.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/riscv/pr115387-1.c: Move to...
> 	* gcc.dg/torture/pr115387-1.c: ...here.
> 	* gcc.target/riscv/pr115387-2.c: Move to...
> 	* gcc.dg/torture/pr115387-2.c: ...here.
OK
jeff


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v1] Test: Move target independent test cases to gcc.dg/torture
  2024-06-11  6:19 [PATCH v1] Test: Move target independent test cases to gcc.dg/torture pan2.li
  2024-06-11 13:40 ` Jeff Law
@ 2024-06-11 13:45 ` Andrew Pinski
  2024-06-11 13:54   ` Li, Pan2
  2024-06-11 14:53 ` [PATCH v2] " pan2.li
  2 siblings, 1 reply; 7+ messages in thread
From: Andrew Pinski @ 2024-06-11 13:45 UTC (permalink / raw)
  To: pan2.li
  Cc: GCC Patches, 钟居哲,
	Kito Cheng, Richard Guenther, Jeff Law

[-- Attachment #1: Type: text/plain, Size: 2359 bytes --]

On Mon, Jun 10, 2024, 11:20 PM <pan2.li@intel.com> wrote:

> From: Pan Li <pan2.li@intel.com>
>
> The test cases of pr115387 are target independent,  at least x86
> and riscv are able to reproduce.  Thus,  move these cases to
> the gcc.dg/torture.
>
> The below test suites are passed.
> 1. The rv64gcv fully regression test.
> 2. The x86 fully regression test.
>
> gcc/testsuite/ChangeLog:
>
>         * gcc.target/riscv/pr115387-1.c: Move to...
>         * gcc.dg/torture/pr115387-1.c: ...here.
>         * gcc.target/riscv/pr115387-2.c: Move to...
>         * gcc.dg/torture/pr115387-2.c: ...here.
>
> Signed-off-by: Pan Li <pan2.li@intel.com>
> ---
>  gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c | 2 +-
>  gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>  rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c
> (92%)
>  rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c
> (84%)
>
> diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-1.c
> b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
> similarity index 92%
> rename from gcc/testsuite/gcc.target/riscv/pr115387-1.c
> rename to gcc/testsuite/gcc.dg/torture/pr115387-1.c
> index a1c926977c4..fde79f66757 100644
> --- a/gcc/testsuite/gcc.target/riscv/pr115387-1.c
> +++ b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
> @@ -1,6 +1,6 @@
>  /* Test there is no ICE when compile.  */
>  /* { dg-do compile } */
> -/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
> +/* { dg-options "-O3" } */
>

Since you are moving it to torture, please remove -O3 as it is already
supplied there as one of the torture options.



>  #define PRINTF_CHK 0x34
>
> diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-2.c
> b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
> similarity index 84%
> rename from gcc/testsuite/gcc.target/riscv/pr115387-2.c
> rename to gcc/testsuite/gcc.dg/torture/pr115387-2.c
> index 7183bf18dfd..0cb4b48d27b 100644
> --- a/gcc/testsuite/gcc.target/riscv/pr115387-2.c
> +++ b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
> @@ -1,6 +1,6 @@
>  /* Test there is no ICE when compile.  */
>  /* { dg-do compile } */
> -/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
> +/* { dg-options "-O3" } */
>
>  #include <stddef.h>
>  #include <stdint-gcc.h>
> --
> 2.34.1
>
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: [PATCH v1] Test: Move target independent test cases to gcc.dg/torture
  2024-06-11 13:45 ` Andrew Pinski
@ 2024-06-11 13:54   ` Li, Pan2
  0 siblings, 0 replies; 7+ messages in thread
From: Li, Pan2 @ 2024-06-11 13:54 UTC (permalink / raw)
  To: Andrew Pinski
  Cc: GCC Patches, 钟居哲,
	Kito Cheng, Richard Guenther, Jeff Law

[-- Attachment #1: Type: text/plain, Size: 2964 bytes --]

> Since you are moving it to torture, please remove -O3 as it is already supplied there as one of the torture options.

Oh, I see. Thanks for comments, and will update it in v2.

Pan

From: Andrew Pinski <pinskia@gmail.com>
Sent: Tuesday, June 11, 2024 9:45 PM
To: Li, Pan2 <pan2.li@intel.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>; 钟居哲 <juzhe.zhong@rivai.ai>; Kito Cheng <kito.cheng@gmail.com>; Richard Guenther <richard.guenther@gmail.com>; Jeff Law <jeffreyalaw@gmail.com>
Subject: Re: [PATCH v1] Test: Move target independent test cases to gcc.dg/torture


On Mon, Jun 10, 2024, 11:20 PM <pan2.li@intel.com<mailto:pan2.li@intel.com>> wrote:
From: Pan Li <pan2.li@intel.com<mailto:pan2.li@intel.com>>

The test cases of pr115387 are target independent,  at least x86
and riscv are able to reproduce.  Thus,  move these cases to
the gcc.dg/torture.

The below test suites are passed.
1. The rv64gcv fully regression test.
2. The x86 fully regression test.

gcc/testsuite/ChangeLog:

        * gcc.target/riscv/pr115387-1.c: Move to...
        * gcc.dg/torture/pr115387-1.c: ...here.
        * gcc.target/riscv/pr115387-2.c: Move to...
        * gcc.dg/torture/pr115387-2.c: ...here.

Signed-off-by: Pan Li <pan2.li@intel.com<mailto:pan2.li@intel.com>>
---
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c | 2 +-
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c (92%)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c (84%)

diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-1.c b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
similarity index 92%
rename from gcc/testsuite/gcc.target/riscv/pr115387-1.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-1.c
index a1c926977c4..fde79f66757 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-1.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
@@ -1,6 +1,6 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
+/* { dg-options "-O3" } */

Since you are moving it to torture, please remove -O3 as it is already supplied there as one of the torture options.



 #define PRINTF_CHK 0x34

diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-2.c b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
similarity index 84%
rename from gcc/testsuite/gcc.target/riscv/pr115387-2.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-2.c
index 7183bf18dfd..0cb4b48d27b 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-2.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
@@ -1,6 +1,6 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
+/* { dg-options "-O3" } */

 #include <stddef.h>
 #include <stdint-gcc.h>
--
2.34.1

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v2] Test: Move target independent test cases to gcc.dg/torture
  2024-06-11  6:19 [PATCH v1] Test: Move target independent test cases to gcc.dg/torture pan2.li
  2024-06-11 13:40 ` Jeff Law
  2024-06-11 13:45 ` Andrew Pinski
@ 2024-06-11 14:53 ` pan2.li
  2024-06-12 18:10   ` Jeff Law
  2 siblings, 1 reply; 7+ messages in thread
From: pan2.li @ 2024-06-11 14:53 UTC (permalink / raw)
  To: gcc-patches
  Cc: juzhe.zhong, kito.cheng, richard.guenther, jeffreyalaw, pinskia, Pan Li

From: Pan Li <pan2.li@intel.com>

The test cases of pr115387 are target independent,  at least x86
and riscv are able to reproduce.  Thus,  move these cases to
the gcc.dg/torture.

The below test suites are passed.
1. The rv64gcv fully regression test.
2. The x86 fully regression test.

gcc/testsuite/ChangeLog:

	* gcc.target/riscv/pr115387-1.c: Move to...
	* gcc.dg/torture/pr115387-1.c: ...here.
	* gcc.target/riscv/pr115387-2.c: Move to...
	* gcc.dg/torture/pr115387-2.c: ...here.

Signed-off-by: Pan Li <pan2.li@intel.com>
---
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c | 1 -
 gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c | 1 -
 2 files changed, 2 deletions(-)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-1.c (92%)
 rename gcc/testsuite/{gcc.target/riscv => gcc.dg/torture}/pr115387-2.c (84%)

diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-1.c b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
similarity index 92%
rename from gcc/testsuite/gcc.target/riscv/pr115387-1.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-1.c
index a1c926977c4..d94e935fade 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-1.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-1.c
@@ -1,6 +1,5 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
 
 #define PRINTF_CHK 0x34
 
diff --git a/gcc/testsuite/gcc.target/riscv/pr115387-2.c b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
similarity index 84%
rename from gcc/testsuite/gcc.target/riscv/pr115387-2.c
rename to gcc/testsuite/gcc.dg/torture/pr115387-2.c
index 7183bf18dfd..9e93024b45c 100644
--- a/gcc/testsuite/gcc.target/riscv/pr115387-2.c
+++ b/gcc/testsuite/gcc.dg/torture/pr115387-2.c
@@ -1,6 +1,5 @@
 /* Test there is no ICE when compile.  */
 /* { dg-do compile } */
-/* { dg-options "-march=rv64gcv -mabi=lp64d -O3" } */
 
 #include <stddef.h>
 #include <stdint-gcc.h>
-- 
2.34.1


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v2] Test: Move target independent test cases to gcc.dg/torture
  2024-06-11 14:53 ` [PATCH v2] " pan2.li
@ 2024-06-12 18:10   ` Jeff Law
  2024-06-13  2:22     ` Li, Pan2
  0 siblings, 1 reply; 7+ messages in thread
From: Jeff Law @ 2024-06-12 18:10 UTC (permalink / raw)
  To: pan2.li, gcc-patches; +Cc: juzhe.zhong, kito.cheng, richard.guenther, pinskia



On 6/11/24 8:53 AM, pan2.li@intel.com wrote:
> From: Pan Li <pan2.li@intel.com>
> 
> The test cases of pr115387 are target independent,  at least x86
> and riscv are able to reproduce.  Thus,  move these cases to
> the gcc.dg/torture.
> 
> The below test suites are passed.
> 1. The rv64gcv fully regression test.
> 2. The x86 fully regression test.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/riscv/pr115387-1.c: Move to...
> 	* gcc.dg/torture/pr115387-1.c: ...here.
> 	* gcc.target/riscv/pr115387-2.c: Move to...
> 	* gcc.dg/torture/pr115387-2.c: ...here.
OK
jeff


^ permalink raw reply	[flat|nested] 7+ messages in thread

* RE: [PATCH v2] Test: Move target independent test cases to gcc.dg/torture
  2024-06-12 18:10   ` Jeff Law
@ 2024-06-13  2:22     ` Li, Pan2
  0 siblings, 0 replies; 7+ messages in thread
From: Li, Pan2 @ 2024-06-13  2:22 UTC (permalink / raw)
  To: Jeff Law, gcc-patches; +Cc: juzhe.zhong, kito.cheng, richard.guenther, pinskia

Committed, thanks Jeff.

Pan

-----Original Message-----
From: Jeff Law <jeffreyalaw@gmail.com> 
Sent: Thursday, June 13, 2024 2:11 AM
To: Li, Pan2 <pan2.li@intel.com>; gcc-patches@gcc.gnu.org
Cc: juzhe.zhong@rivai.ai; kito.cheng@gmail.com; richard.guenther@gmail.com; pinskia@gmail.com
Subject: Re: [PATCH v2] Test: Move target independent test cases to gcc.dg/torture



On 6/11/24 8:53 AM, pan2.li@intel.com wrote:
> From: Pan Li <pan2.li@intel.com>
> 
> The test cases of pr115387 are target independent,  at least x86
> and riscv are able to reproduce.  Thus,  move these cases to
> the gcc.dg/torture.
> 
> The below test suites are passed.
> 1. The rv64gcv fully regression test.
> 2. The x86 fully regression test.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/riscv/pr115387-1.c: Move to...
> 	* gcc.dg/torture/pr115387-1.c: ...here.
> 	* gcc.target/riscv/pr115387-2.c: Move to...
> 	* gcc.dg/torture/pr115387-2.c: ...here.
OK
jeff


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-06-13  2:22 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-06-11  6:19 [PATCH v1] Test: Move target independent test cases to gcc.dg/torture pan2.li
2024-06-11 13:40 ` Jeff Law
2024-06-11 13:45 ` Andrew Pinski
2024-06-11 13:54   ` Li, Pan2
2024-06-11 14:53 ` [PATCH v2] " pan2.li
2024-06-12 18:10   ` Jeff Law
2024-06-13  2:22     ` Li, Pan2

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).