Re: [PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system

2023-04-05 Thread Jeff Law via Gcc-patches
On 4/2/23 18:38, juzhe.zh...@rivai.ai wrote: From: Ju-Zhe Zhong It's quite obvious that the order of vrsub SEW64 is wrong. gcc/ChangeLog: * config/riscv/vector.md: Fix incorrect operand order. gcc/testsuite/ChangeLog: * g++.target/riscv/rvv/base/bug-23.C: New test. Tha

Re: [PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system

2023-04-04 Thread Jeff Law via Gcc-patches
On 4/2/23 22:18, juzhe.zh...@rivai.ai wrote: OK, would you mind telling me some tool can help for formatting of .MD file? Since clang-format only works on .cc/.h file. No idea. I don't use clang-format. But I've also had 30+ years in this codebase, so most of its coding conventions are one

Re: Re: [PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system

2023-04-02 Thread juzhe.zh...@rivai.ai
s. Thanks. juzhe.zh...@rivai.ai From: Jeff Law Date: 2023-04-03 12:13 To: juzhe.zhong; gcc-patches CC: kito.cheng; palmer Subject: Re: [PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system On 4/2/23 18:38, juzhe.zh...@rivai.ai wrote: > From: Ju-Zhe Zhong > > It's quit

Re: [PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system

2023-04-02 Thread Jeff Law via Gcc-patches
On 4/2/23 18:38, juzhe.zh...@rivai.ai wrote: From: Ju-Zhe Zhong It's quite obvious that the order of vrsub SEW64 is wrong. gcc/ChangeLog: * config/riscv/vector.md: Fix incorrect operand order. Note the formatting in that code is wrong on multiple levels. While I don't typically

[PATCH] RISC-V: Fix SEW64 of vrsub.vx runtime fail in RV32 system

2023-04-02 Thread juzhe . zhong
From: Ju-Zhe Zhong It's quite obvious that the order of vrsub SEW64 is wrong. gcc/ChangeLog: * config/riscv/vector.md: Fix incorrect operand order. gcc/testsuite/ChangeLog: * g++.target/riscv/rvv/base/bug-23.C: New test. --- gcc/config/riscv/vector.md|