RISC-V: Force ilp32d for the T-Head FMV test

Message ID 20230411201031.22067-1-palmer@rivosinc.com
State Committed
Commit c16848ed8f30be952ac0167fd464ae794fa5ac67
Headers
Series RISC-V: Force ilp32d for the T-Head FMV test |

Commit Message

Palmer Dabbelt April 11, 2023, 8:10 p.m. UTC
  These functions are NOPs on the soft-float ABIs.  Since we're already
forcing the ISA, let's just force the ABI too.

gcc/testsuite/ChangeLog:

	* gcc.target/riscv/xtheadfmv-fmv.c: Force the ilp32d ABI.
---
This one is also in the testing queue.  OK for trunk?
---
 gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Jeff Law April 11, 2023, 9:01 p.m. UTC | #1
On 4/11/23 14:10, Palmer Dabbelt wrote:
> These functions are NOPs on the soft-float ABIs.  Since we're already
> forcing the ISA, let's just force the ABI too.
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/riscv/xtheadfmv-fmv.c: Force the ilp32d ABI.
> ---
> This one is also in the testing queue.  OK for trunk?
OK
jeff
  
Palmer Dabbelt April 17, 2023, 7:19 p.m. UTC | #2
On Tue, 11 Apr 2023 14:01:35 PDT (-0700), gcc-patches@gcc.gnu.org wrote:
>
>
> On 4/11/23 14:10, Palmer Dabbelt wrote:
>> These functions are NOPs on the soft-float ABIs.  Since we're already
>> forcing the ISA, let's just force the ABI too.
>>
>> gcc/testsuite/ChangeLog:
>>
>> 	* gcc.target/riscv/xtheadfmv-fmv.c: Force the ilp32d ABI.
>> ---
>> This one is also in the testing queue.  OK for trunk?
> OK

Committed.
  

Patch

diff --git a/gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c b/gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c
index 10d035e9e1d..1036044291e 100644
--- a/gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c
+++ b/gcc/testsuite/gcc.target/riscv/xtheadfmv-fmv.c
@@ -1,5 +1,5 @@ 
 /* { dg-do compile { target { rv32 } } } */
-/* { dg-options "-march=rv32gc_xtheadfmv" } */
+/* { dg-options "-march=rv32gc_xtheadfmv -mabi=ilp32d" } */
 /* { dg-skip-if "" { *-*-* } { "-O0" } } */
 
 double