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)
            {

Reply via email to