public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] RISC-V: Remove tail && mask policy operand for vmclr,vmset,vmld,vmst
@ 2022-11-28 14:21 juzhe.zhong
  2022-11-28 16:48 ` [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst Jeff Law
  0 siblings, 1 reply; 3+ messages in thread
From: juzhe.zhong @ 2022-11-28 14:21 UTC (permalink / raw)
  To: gcc-patches; +Cc: kito.cheng, palmer, Ju-Zhe Zhong

From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>

Since mask instruction doesn't need policy, so remove it to make it look reasonable.
gcc/ChangeLog:

        * config/riscv/vector.md: Remove TA && MA operands.

---
 gcc/config/riscv/vector.md | 2 --
 1 file changed, 2 deletions(-)

diff --git a/gcc/config/riscv/vector.md b/gcc/config/riscv/vector.md
index 3bb87232d3f..38da2f7f095 100644
--- a/gcc/config/riscv/vector.md
+++ b/gcc/config/riscv/vector.md
@@ -593,8 +593,6 @@
 	  (unspec:VB
 	    [(match_operand:VB 1 "vector_mask_operand"   "Wc1, Wc1, Wc1, Wc1, Wc1")
 	     (match_operand 4 "vector_length_operand"    " rK,  rK,  rK,  rK,  rK")
-	     (match_operand 5 "const_int_operand"        "  i,   i,   i,   i,   i")
-	     (match_operand 6 "const_int_operand"        "  i,   i,   i,   i,   i")
 	     (reg:SI VL_REGNUM)
 	     (reg:SI VTYPE_REGNUM)] UNSPEC_VPREDICATE)
 	  (match_operand:VB 3 "vector_move_operand"      "  m,  vr,  vr, Wc0, Wc1")
-- 
2.36.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst
  2022-11-28 14:21 [PATCH] RISC-V: Remove tail && mask policy operand for vmclr,vmset,vmld,vmst juzhe.zhong
@ 2022-11-28 16:48 ` Jeff Law
  2022-11-28 22:52   ` 钟居哲
  0 siblings, 1 reply; 3+ messages in thread
From: Jeff Law @ 2022-11-28 16:48 UTC (permalink / raw)
  To: juzhe.zhong, gcc-patches; +Cc: kito.cheng, palmer


On 11/28/22 07:21, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
>
> Since mask instruction doesn't need policy, so remove it to make it look reasonable.
> gcc/ChangeLog:
>
>          * config/riscv/vector.md: Remove TA && MA operands.

Does this fix a known bug or is it just a cleanup?   I think the latter, 
but I want to be sure.



Jeff



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Re: [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst
  2022-11-28 16:48 ` [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst Jeff Law
@ 2022-11-28 22:52   ` 钟居哲
  0 siblings, 0 replies; 3+ messages in thread
From: 钟居哲 @ 2022-11-28 22:52 UTC (permalink / raw)
  To: Jeff Law, gcc-patches; +Cc: kito.cheng, palmer

[-- Attachment #1: Type: text/plain, Size: 639 bytes --]

Yes, it's a cleanup.



juzhe.zhong@rivai.ai
 
From: Jeff Law
Date: 2022-11-29 00:48
To: juzhe.zhong; gcc-patches
CC: kito.cheng; palmer
Subject: Re: [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst
 
On 11/28/22 07:21, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
>
> Since mask instruction doesn't need policy, so remove it to make it look reasonable.
> gcc/ChangeLog:
>
>          * config/riscv/vector.md: Remove TA && MA operands.
 
Does this fix a known bug or is it just a cleanup?   I think the latter, 
but I want to be sure.
 
 
 
Jeff
 
 
 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-11-28 22:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-28 14:21 [PATCH] RISC-V: Remove tail && mask policy operand for vmclr,vmset,vmld,vmst juzhe.zhong
2022-11-28 16:48 ` [PATCH] RISC-V: Remove tail && mask policy operand for vmclr, vmset, vmld, vmst Jeff Law
2022-11-28 22:52   ` 钟居哲

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).