diff mbox series

RISC-V: Add -fno-vect-cost-model to pr112773 testcase

Message ID 20231214213201.180346-1-patrick@rivosinc.com
State Unresolved
Headers
Series RISC-V: Add -fno-vect-cost-model to pr112773 testcase |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

Patrick O'Neill Dec. 14, 2023, 9:32 p.m. UTC 
  The testcase for pr112773 started passing after r14-6472-g8501edba91e
which was before the actual fix. This patch adds -fno-vect-cost-model
which prevents the testcase from passing due to the vls change.

gcc/testsuite/ChangeLog:

	* gcc.target/riscv/rvv/autovec/partial/pr112773.c: Add
	-fno-vect-cost-model.

Signed-off-by: Patrick O'Neill <patrick@rivosinc.com>
---
 gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

juzhe.zhong@rivai.ai Dec. 14, 2023, 10:07 p.m. UTC | #1 
LGTM



juzhe.zhong@rivai.ai
 
From: Patrick O'Neill
Date: 2023-12-15 05:32
To: gcc-patches
CC: rdapp.gcc; juzhe.zhong; Patrick O'Neill
Subject: [PATCH] RISC-V: Add -fno-vect-cost-model to pr112773 testcase
The testcase for pr112773 started passing after r14-6472-g8501edba91e
which was before the actual fix. This patch adds -fno-vect-cost-model
which prevents the testcase from passing due to the vls change.
 
gcc/testsuite/ChangeLog:
 
* gcc.target/riscv/rvv/autovec/partial/pr112773.c: Add
-fno-vect-cost-model.
 
Signed-off-by: Patrick O'Neill <patrick@rivosinc.com>
---
gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
 
diff --git a/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c b/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
index 5f7374b0040..57104c9ebec 100644
--- a/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
+++ b/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
@@ -1,5 +1,5 @@
/* { dg-do compile } */
-/* { dg-additional-options "-march=rv32gcv_zvl256b -mabi=ilp32d -O3" } */
+/* { dg-additional-options "-march=rv32gcv_zvl256b -mabi=ilp32d -O3 -fno-vect-cost-model" } */
long long a;
int b, c;
-- 
2.42.0
Jeff Law Dec. 15, 2023, 9:47 p.m. UTC | #2 
On 12/14/23 14:32, Patrick O'Neill wrote:
> The testcase for pr112773 started passing after r14-6472-g8501edba91e
> which was before the actual fix. This patch adds -fno-vect-cost-model
> which prevents the testcase from passing due to the vls change.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/riscv/rvv/autovec/partial/pr112773.c: Add
> 	-fno-vect-cost-model.
> 
> Signed-off-by: Patrick O'Neill <patrick@rivosinc.com>
I've pushed this to the trunk.

jeff
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c b/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
index 5f7374b0040..57104c9ebec 100644
--- a/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
+++ b/gcc/testsuite/gcc.target/riscv/rvv/autovec/partial/pr112773.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile } */
-/* { dg-additional-options "-march=rv32gcv_zvl256b -mabi=ilp32d -O3" } */
+/* { dg-additional-options "-march=rv32gcv_zvl256b -mabi=ilp32d -O3 -fno-vect-cost-model" } */
 
 long long a;
 int b, c;