public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc(refs/vendors/riscv/heads/gcc-13-with-riscv-opts)] RISC-V: Rename some variables of vector_block_info[NFC]
@ 2023-10-23 2:12 Jeff Law
0 siblings, 0 replies; only message in thread
From: Jeff Law @ 2023-10-23 2:12 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:b1f09ad1a6772434ccf512b027ac6554d4b04a4c
commit b1f09ad1a6772434ccf512b027ac6554d4b04a4c
Author: Juzhe-Zhong <juzhe.zhong@rivai.ai>
Date: Fri Oct 20 16:23:18 2023 +0800
RISC-V: Rename some variables of vector_block_info[NFC]
1. Remove "m_" prefix as they are not private members.
2. Rename infos -> local_infos, info -> global_info to clarify their meaning.
Pushed as it is obvious.
gcc/ChangeLog:
* config/riscv/riscv-vsetvl.cc (pre_vsetvl::fuse_local_vsetvl_info): Rename variables.
(pre_vsetvl::pre_global_vsetvl_info): Ditto.
(pre_vsetvl::emit_vsetvl): Ditto.
(cherry picked from commit 4fd09aed384df6acb141ee1f319a500c239626a4)
Diff:
---
gcc/config/riscv/riscv-vsetvl.cc | 57 ++++++++++++++++++++--------------------
1 file changed, 29 insertions(+), 28 deletions(-)
diff --git a/gcc/config/riscv/riscv-vsetvl.cc b/gcc/config/riscv/riscv-vsetvl.cc
index e136351aee5f..5f26a9ec96a8 100644
--- a/gcc/config/riscv/riscv-vsetvl.cc
+++ b/gcc/config/riscv/riscv-vsetvl.cc
@@ -1281,55 +1281,56 @@ public:
/* The static execute probability of the demand info. */
profile_probability probability;
- auto_vec<vsetvl_info> infos;
- vsetvl_info m_info;
- bb_info *m_bb;
+ auto_vec<vsetvl_info> local_infos;
+ vsetvl_info global_info;
+ bb_info *bb;
bool full_available;
- vsetvl_block_info () : m_bb (nullptr), full_available (false)
+ vsetvl_block_info () : bb (nullptr), full_available (false)
{
- infos.safe_grow_cleared (0);
- m_info.set_empty ();
+ local_infos.safe_grow_cleared (0);
+ global_info.set_empty ();
}
vsetvl_block_info (const vsetvl_block_info &other)
- : probability (other.probability), infos (other.infos.copy ()),
- m_info (other.m_info), m_bb (other.m_bb)
+ : probability (other.probability), local_infos (other.local_infos.copy ()),
+ global_info (other.global_info), bb (other.bb)
{}
vsetvl_info &get_entry_info ()
{
gcc_assert (!empty_p ());
- return infos.is_empty () ? m_info : infos[0];
+ return local_infos.is_empty () ? global_info : local_infos[0];
}
vsetvl_info &get_exit_info ()
{
gcc_assert (!empty_p ());
- return infos.is_empty () ? m_info : infos[infos.length () - 1];
+ return local_infos.is_empty () ? global_info
+ : local_infos[local_infos.length () - 1];
}
const vsetvl_info &get_entry_info () const
{
gcc_assert (!empty_p ());
- return infos.is_empty () ? m_info : infos[0];
+ return local_infos.is_empty () ? global_info : local_infos[0];
}
const vsetvl_info &get_exit_info () const
{
gcc_assert (!empty_p ());
- return infos.is_empty () ? m_info : infos[infos.length () - 1];
+ return local_infos.is_empty () ? global_info
+ : local_infos[local_infos.length () - 1];
}
- bool empty_p () const { return infos.is_empty () && !has_info (); }
- bool has_info () const { return !m_info.empty_p (); }
+ bool empty_p () const { return local_infos.is_empty () && !has_info (); }
+ bool has_info () const { return !global_info.empty_p (); }
void set_info (const vsetvl_info &info)
{
- gcc_assert (infos.is_empty ());
- m_info = info;
- m_info.set_bb (m_bb);
+ gcc_assert (local_infos.is_empty ());
+ global_info = info;
+ global_info.set_bb (bb);
}
- void set_empty_info () { m_info.set_empty (); }
+ void set_empty_info () { global_info.set_empty (); }
};
-
/* Demand system is the RVV-based VSETVL info analysis tools wrapper.
It defines compatible rules for SEW/LMUL, POLICY and AVL.
Also, it provides 3 iterfaces avaiable_p, compatible_p and
@@ -2323,7 +2324,7 @@ public:
block_info.get_entry_info ().dump (file, " ");
fprintf (file, " Footer vsetvl info:");
block_info.get_exit_info ().dump (file, " ");
- for (const auto &info : block_info.infos)
+ for (const auto &info : block_info.local_infos)
{
fprintf (file,
" insn %d vsetvl info:", info.get_insn ()->uid ());
@@ -2740,7 +2741,7 @@ pre_vsetvl::fuse_local_vsetvl_info ()
for (bb_info *bb : crtl->ssa->bbs ())
{
auto &block_info = get_block_info (bb);
- block_info.m_bb = bb;
+ block_info.bb = bb;
if (dump_file && (dump_flags & TDF_DETAILS))
{
fprintf (dump_file, " Try fuse basic block %d\n", bb->index ());
@@ -2768,7 +2769,7 @@ pre_vsetvl::fuse_local_vsetvl_info ()
else if ((curr_info.unknown_p () && prev_info.valid_p ())
|| (curr_info.valid_p () && prev_info.unknown_p ()))
{
- block_info.infos.safe_push (prev_info);
+ block_info.local_infos.safe_push (prev_info);
prev_info = curr_info;
}
else if (curr_info.valid_p () && prev_info.valid_p ())
@@ -2825,14 +2826,14 @@ pre_vsetvl::fuse_local_vsetvl_info ()
fprintf (dump_file, " curr_info: ");
curr_info.dump (dump_file, " ");
}
- block_info.infos.safe_push (prev_info);
+ block_info.local_infos.safe_push (prev_info);
prev_info = curr_info;
}
}
}
if (prev_info.valid_p () || prev_info.unknown_p ())
- block_info.infos.safe_push (prev_info);
+ block_info.local_infos.safe_push (prev_info);
}
m_avl_regs = sbitmap_alloc (GP_REG_LAST + 1);
@@ -3194,9 +3195,9 @@ pre_vsetvl::pre_global_vsetvl_info ()
vsetvl_info &curr_info = block_info.get_entry_info ();
if (curr_info.delete_p ())
{
- if (block_info.infos.is_empty ())
+ if (block_info.local_infos.is_empty ())
continue;
- curr_info = block_info.infos[0];
+ curr_info = block_info.local_infos[0];
}
if (curr_info.valid_p () && !curr_info.vl_use_by_non_rvv_insn_p ()
&& preds_has_same_avl_p (curr_info))
@@ -3204,7 +3205,7 @@ pre_vsetvl::pre_global_vsetvl_info ()
vsetvl_info prev_info = vsetvl_info ();
prev_info.set_empty ();
- for (auto &curr_info : block_info.infos)
+ for (auto &curr_info : block_info.local_infos)
{
if (prev_info.valid_p () && curr_info.valid_p ()
&& m_dem.avl_available_p (prev_info, curr_info))
@@ -3221,7 +3222,7 @@ pre_vsetvl::emit_vsetvl ()
for (const bb_info *bb : crtl->ssa->bbs ())
{
- for (const auto &curr_info : get_block_info (bb).infos)
+ for (const auto &curr_info : get_block_info (bb).local_infos)
{
insn_info *insn = curr_info.get_insn ();
if (curr_info.delete_p ())
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2023-10-23 2:12 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-23 2:12 [gcc(refs/vendors/riscv/heads/gcc-13-with-riscv-opts)] RISC-V: Rename some variables of vector_block_info[NFC] Jeff Law
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).