* [PATCH] RISC-V: Fix muti-line condition format
@ 2022-12-19 23:09 juzhe.zhong
2022-12-20 0:27 ` Jeff Law
0 siblings, 1 reply; 3+ messages in thread
From: juzhe.zhong @ 2022-12-19 23:09 UTC (permalink / raw)
To: gcc-patches; +Cc: kito.cheng, palmer, jeffreyalaw, Ju-Zhe Zhong
From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
gcc/ChangeLog:
* config/riscv/riscv-vsetvl.cc (vlmax_avl_insn_p): Fix multi-line conditional.
(vsetvl_insn_p): Ditto.
(same_bb_and_before_p): Ditto.
(same_bb_and_after_or_equal_p): Ditto.
---
gcc/config/riscv/riscv-vsetvl.cc | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/gcc/config/riscv/riscv-vsetvl.cc b/gcc/config/riscv/riscv-vsetvl.cc
index 3ca3fc15e5a..0c2ff630e96 100644
--- a/gcc/config/riscv/riscv-vsetvl.cc
+++ b/gcc/config/riscv/riscv-vsetvl.cc
@@ -112,8 +112,8 @@ vlmax_avl_p (rtx x)
static bool
vlmax_avl_insn_p (rtx_insn *rinsn)
{
- return INSN_CODE (rinsn) == CODE_FOR_vlmax_avlsi
- || INSN_CODE (rinsn) == CODE_FOR_vlmax_avldi;
+ return (INSN_CODE (rinsn) == CODE_FOR_vlmax_avlsi
+ || INSN_CODE (rinsn) == CODE_FOR_vlmax_avldi);
}
static bool
@@ -156,24 +156,24 @@ vector_config_insn_p (rtx_insn *rinsn)
static bool
vsetvl_insn_p (rtx_insn *rinsn)
{
- return INSN_CODE (rinsn) == CODE_FOR_vsetvldi
- || INSN_CODE (rinsn) == CODE_FOR_vsetvlsi;
+ return (INSN_CODE (rinsn) == CODE_FOR_vsetvldi
+ || INSN_CODE (rinsn) == CODE_FOR_vsetvlsi);
}
/* Return true if INSN1 comes befeore INSN2 in the same block. */
static bool
same_bb_and_before_p (const insn_info *insn1, const insn_info *insn2)
{
- return (insn1->bb ()->index () == insn2->bb ()->index ())
- && (*insn1 < *insn2);
+ return ((insn1->bb ()->index () == insn2->bb ()->index ())
+ && (*insn1 < *insn2));
}
/* Return true if INSN1 comes after or equal INSN2 in the same block. */
static bool
same_bb_and_after_or_equal_p (const insn_info *insn1, const insn_info *insn2)
{
- return (insn1->bb ()->index () == insn2->bb ()->index ())
- && (*insn1 >= *insn2);
+ return ((insn1->bb ()->index () == insn2->bb ()->index ())
+ && (*insn1 >= *insn2));
}
/* An "anticipatable occurrence" is one that is the first occurrence in the
--
2.36.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] RISC-V: Fix muti-line condition format
2022-12-19 23:09 [PATCH] RISC-V: Fix muti-line condition format juzhe.zhong
@ 2022-12-20 0:27 ` Jeff Law
2022-12-23 5:43 ` Kito Cheng
0 siblings, 1 reply; 3+ messages in thread
From: Jeff Law @ 2022-12-20 0:27 UTC (permalink / raw)
To: juzhe.zhong, gcc-patches; +Cc: kito.cheng, palmer
On 12/19/22 16:09, juzhe.zhong@rivai.ai wrote:
> From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
>
> gcc/ChangeLog:
>
> * config/riscv/riscv-vsetvl.cc (vlmax_avl_insn_p): Fix multi-line conditional.
> (vsetvl_insn_p): Ditto.
> (same_bb_and_before_p): Ditto.
> (same_bb_and_after_or_equal_p): Ditto.
OK
jeff
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] RISC-V: Fix muti-line condition format
2022-12-20 0:27 ` Jeff Law
@ 2022-12-23 5:43 ` Kito Cheng
0 siblings, 0 replies; 3+ messages in thread
From: Kito Cheng @ 2022-12-23 5:43 UTC (permalink / raw)
To: Jeff Law; +Cc: juzhe.zhong, gcc-patches, palmer
Committed, thanks :)
On Tue, Dec 20, 2022 at 8:28 AM Jeff Law via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>
>
>
> On 12/19/22 16:09, juzhe.zhong@rivai.ai wrote:
> > From: Ju-Zhe Zhong <juzhe.zhong@rivai.ai>
> >
> > gcc/ChangeLog:
> >
> > * config/riscv/riscv-vsetvl.cc (vlmax_avl_insn_p): Fix multi-line conditional.
> > (vsetvl_insn_p): Ditto.
> > (same_bb_and_before_p): Ditto.
> > (same_bb_and_after_or_equal_p): Ditto.
> OK
> jeff
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-12-23 5:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-19 23:09 [PATCH] RISC-V: Fix muti-line condition format juzhe.zhong
2022-12-20 0:27 ` Jeff Law
2022-12-23 5:43 ` Kito Cheng
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).