RE: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll
Committed, thanks Kito and Juzhe. Pan -Original Message- From: Kito Cheng Sent: Thursday, April 25, 2024 2:24 PM To: juzhe.zh...@rivai.ai Cc: Li, Pan2 ; gcc-patches ; Robin Dapp Subject: Re: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll LGTM, thanks :) On Thu, Apr 25, 2024 at 9:26 AM juzhe.zh...@rivai.ai wrote: > > lgtm > > > juzhe.zh...@rivai.ai > > > From: pan2.li > Date: 2024-04-25 09:25 > To: gcc-patches > CC: juzhe.zhong; kito.cheng; rdapp.gcc; Pan Li > Subject: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll > From: Pan Li > > We missed the existing early clobber for the dest operand of vwsll > pattern when resolve the conflict of revert register overlap. Thus > add it back to the pattern. Unfortunately, we have no test to cover > this part and will improve this after GCC-15 open. > > The below tests are passed for this patch: > * The rv64gcv fully regression test with isl build. > > gcc/ChangeLog: > > * config/riscv/vector-crypto.md: Add early clobber to the > dest operand of vwsll. > > Signed-off-by: Pan Li > --- > gcc/config/riscv/vector-crypto.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/config/riscv/vector-crypto.md > b/gcc/config/riscv/vector-crypto.md > index 8a4888a7653..e474ddf5da7 100755 > --- a/gcc/config/riscv/vector-crypto.md > +++ b/gcc/config/riscv/vector-crypto.md > @@ -303,7 +303,7 @@ (define_insn "@pred_vwsll" > (set_attr "mode" "")]) > (define_insn "@pred_vwsll_scalar" > - [(set (match_operand:VWEXTI 0 "register_operand" "=vr, > vr") > + [(set (match_operand:VWEXTI 0 "register_operand" "=, > ") > (if_then_else:VWEXTI > (unspec: > [(match_operand: 1 "vector_mask_operand" "vmWc1, > vmWc1") > -- > 2.34.1 > >
Re: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll
LGTM, thanks :) On Thu, Apr 25, 2024 at 9:26 AM juzhe.zh...@rivai.ai wrote: > > lgtm > > > juzhe.zh...@rivai.ai > > > From: pan2.li > Date: 2024-04-25 09:25 > To: gcc-patches > CC: juzhe.zhong; kito.cheng; rdapp.gcc; Pan Li > Subject: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll > From: Pan Li > > We missed the existing early clobber for the dest operand of vwsll > pattern when resolve the conflict of revert register overlap. Thus > add it back to the pattern. Unfortunately, we have no test to cover > this part and will improve this after GCC-15 open. > > The below tests are passed for this patch: > * The rv64gcv fully regression test with isl build. > > gcc/ChangeLog: > > * config/riscv/vector-crypto.md: Add early clobber to the > dest operand of vwsll. > > Signed-off-by: Pan Li > --- > gcc/config/riscv/vector-crypto.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/config/riscv/vector-crypto.md > b/gcc/config/riscv/vector-crypto.md > index 8a4888a7653..e474ddf5da7 100755 > --- a/gcc/config/riscv/vector-crypto.md > +++ b/gcc/config/riscv/vector-crypto.md > @@ -303,7 +303,7 @@ (define_insn "@pred_vwsll" > (set_attr "mode" "")]) > (define_insn "@pred_vwsll_scalar" > - [(set (match_operand:VWEXTI 0 "register_operand" "=vr, > vr") > + [(set (match_operand:VWEXTI 0 "register_operand" "=, > ") > (if_then_else:VWEXTI > (unspec: > [(match_operand: 1 "vector_mask_operand" "vmWc1, > vmWc1") > -- > 2.34.1 > >
Re: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll
lgtm juzhe.zh...@rivai.ai From: pan2.li Date: 2024-04-25 09:25 To: gcc-patches CC: juzhe.zhong; kito.cheng; rdapp.gcc; Pan Li Subject: [PATCH v1] RISC-V: Add early clobber to the dest of vwsll From: Pan Li We missed the existing early clobber for the dest operand of vwsll pattern when resolve the conflict of revert register overlap. Thus add it back to the pattern. Unfortunately, we have no test to cover this part and will improve this after GCC-15 open. The below tests are passed for this patch: * The rv64gcv fully regression test with isl build. gcc/ChangeLog: * config/riscv/vector-crypto.md: Add early clobber to the dest operand of vwsll. Signed-off-by: Pan Li --- gcc/config/riscv/vector-crypto.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/riscv/vector-crypto.md b/gcc/config/riscv/vector-crypto.md index 8a4888a7653..e474ddf5da7 100755 --- a/gcc/config/riscv/vector-crypto.md +++ b/gcc/config/riscv/vector-crypto.md @@ -303,7 +303,7 @@ (define_insn "@pred_vwsll" (set_attr "mode" "")]) (define_insn "@pred_vwsll_scalar" - [(set (match_operand:VWEXTI 0 "register_operand" "=vr, vr") + [(set (match_operand:VWEXTI 0 "register_operand" "=,") (if_then_else:VWEXTI (unspec: [(match_operand: 1 "vector_mask_operand" "vmWc1, vmWc1") -- 2.34.1