https://gcc.gnu.org/g:ffb521f157fec823907913602ed5cb73dd1c63e8
commit r15-166-gffb521f157fec823907913602ed5cb73dd1c63e8 Author: Andrew Pinski <quic_apin...@quicinc.com> Date: Sat Apr 27 18:54:43 2024 -0700 Remove m_nloops field from loop_versioning This is a small cleanup of loop_versioning where m_nloops is only used in the constructor so we can remove the whole field. Bootstrapped and tested on x86_64-linux-gnu. gcc/ChangeLog: * gimple-loop-versioning.cc (loop_versioning): Remove m_nloops field. (loop_versioning::loop_versioning): Remove initialization of m_nloops field and move it to be a local variable. (loop_versioning::analyze_blocks): Fix formating. Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com> Diff: --- gcc/gimple-loop-versioning.cc | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/gcc/gimple-loop-versioning.cc b/gcc/gimple-loop-versioning.cc index 17877f06921..adea207659b 100644 --- a/gcc/gimple-loop-versioning.cc +++ b/gcc/gimple-loop-versioning.cc @@ -322,9 +322,6 @@ private: /* An obstack to use for general allocation. */ obstack m_obstack; - /* The number of loops in the function. */ - unsigned int m_nloops; - /* The total number of loop version conditions we've found. */ unsigned int m_num_conditions; @@ -525,10 +522,10 @@ loop_versioning::name_prop::value_of_expr (tree val, gimple *) loop_versioning::loop_versioning (function *fn) : m_fn (fn), - m_nloops (number_of_loops (fn)), m_num_conditions (0), m_address_table (31) { + unsigned m_nloops = number_of_loops (fn); bitmap_obstack_initialize (&m_bitmap_obstack); gcc_obstack_init (&m_obstack); @@ -1437,7 +1434,7 @@ loop_versioning::analyze_blocks () { linfo.rejected_p = true; break; - } + } if (!linfo.rejected_p) {