Sure, will send v2 for this.
Pan
-----Original Message-----
From: Jeff Law <jeffreya...@gmail.com>
Sent: Sunday, August 18, 2024 11:19 PM
To: Li, Pan2 <pan2...@intel.com>; gcc-patches@gcc.gnu.org
Cc: richard.guent...@gmail.com; juzhe.zh...@rivai.ai; kito.ch...@gmail.com;
rdapp....@gmail.com; s...@gentoo.org
Subject: Re: [PATCH v1] Test: Move pr116278 run test to c-torture [NFC]
On 8/18/24 1:13 AM, pan2...@intel.com wrote:
> From: Pan Li <pan2...@intel.com>
>
> Move the run test of pr116278 to c-torture and leave the risc-v the
> asm check under risc-v part.
>
> PR target/116278
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/riscv/pr116278-run-1.c: Take compile instead of
> run test.
> * gcc.target/riscv/pr116278-run-2.c: Ditto.
> * gcc.c-torture/execute/pr116278-run-1.c: New test.
> * gcc.c-torture/execute/pr116278-run-2.c: New test.
We should be using the dg-torture framework, so the right directory for
the test is gcc.dg/torture.
I suspect these tests (just based on the constants that appear) may not
work on the 16 bit integer targets. So we may need
/* { dg-require-effective-target int32 } */
But I don't mind faulting that in if/when we see the 16bit int targets
complain.
So OK in the right directory (gcc.dg/torture).
Jeff
