| Message ID | 20241209061031.205550-1-pan2.li@intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v1] RISC-V: Fix incorrect optimization options passing to partial | expand |
ok On Mon, Dec 9, 2024 at 2:13 PM <pan2.li@intel.com> wrote: > > From: Pan Li <pan2.li@intel.com> > > Like the strided load/store, the testcases of vector partial > are designed to pick up different sorts of optimization options but > actually these option are ignored according to the Execution log of > the gcc.log. > > This patch would like to make it correct almost the same as how we > fixed for strided load/store. > > The below test suites are passed for this patch. > * The rv64gcv fully regression test. > > It is test only patch and obvious up to a point, will commit it > directly if no comments in next 48H. > > gcc/testsuite/ChangeLog: > > * gcc.target/riscv/rvv/rvv.exp: Fix the incorrect optimization > options passing to testcases. > > Signed-off-by: Pan Li <pan2.li@intel.com> > --- > gcc/testsuite/gcc.target/riscv/rvv/rvv.exp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > index 98759865fb5..1278f23e8ea 100644 > --- a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > +++ b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > @@ -61,7 +61,7 @@ set AUTOVEC_TEST_OPTS [list \ > {-ftree-vectorize -O2 -mrvv-max-lmul=dynamic} ] > foreach op $AUTOVEC_TEST_OPTS { > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/partial/*.\[cS\]]] \ > - "" "$op" > + "$op" "" > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/binop/*.\[cS\]]] \ > "$op" "" > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/cmp/*.\[cS\]]] \ > -- > 2.43.0 >
Thanks Kito, all issues like below of rvv.exp are fixed. Pan -----Original Message----- From: Kito Cheng <kito.cheng@gmail.com> Sent: Monday, December 9, 2024 2:58 PM To: Li, Pan2 <pan2.li@intel.com> Cc: gcc-patches@gcc.gnu.org; juzhe.zhong@rivai.ai; jeffreyalaw@gmail.com; rdapp.gcc@gmail.com Subject: Re: [PATCH v1] RISC-V: Fix incorrect optimization options passing to partial ok On Mon, Dec 9, 2024 at 2:13 PM <pan2.li@intel.com> wrote: > > From: Pan Li <pan2.li@intel.com> > > Like the strided load/store, the testcases of vector partial > are designed to pick up different sorts of optimization options but > actually these option are ignored according to the Execution log of > the gcc.log. > > This patch would like to make it correct almost the same as how we > fixed for strided load/store. > > The below test suites are passed for this patch. > * The rv64gcv fully regression test. > > It is test only patch and obvious up to a point, will commit it > directly if no comments in next 48H. > > gcc/testsuite/ChangeLog: > > * gcc.target/riscv/rvv/rvv.exp: Fix the incorrect optimization > options passing to testcases. > > Signed-off-by: Pan Li <pan2.li@intel.com> > --- > gcc/testsuite/gcc.target/riscv/rvv/rvv.exp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > index 98759865fb5..1278f23e8ea 100644 > --- a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > +++ b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp > @@ -61,7 +61,7 @@ set AUTOVEC_TEST_OPTS [list \ > {-ftree-vectorize -O2 -mrvv-max-lmul=dynamic} ] > foreach op $AUTOVEC_TEST_OPTS { > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/partial/*.\[cS\]]] \ > - "" "$op" > + "$op" "" > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/binop/*.\[cS\]]] \ > "$op" "" > dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/cmp/*.\[cS\]]] \ > -- > 2.43.0 >
diff --git a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp index 98759865fb5..1278f23e8ea 100644 --- a/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp +++ b/gcc/testsuite/gcc.target/riscv/rvv/rvv.exp @@ -61,7 +61,7 @@ set AUTOVEC_TEST_OPTS [list \ {-ftree-vectorize -O2 -mrvv-max-lmul=dynamic} ] foreach op $AUTOVEC_TEST_OPTS { dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/partial/*.\[cS\]]] \ - "" "$op" + "$op" "" dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/binop/*.\[cS\]]] \ "$op" "" dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/autovec/cmp/*.\[cS\]]] \