Format code and improve a function name.
authorPat Gavlin <pagavlin@microsoft.com>
Mon, 9 Jan 2017 19:36:28 +0000 (11:36 -0800)
committerPat Gavlin <pagavlin@microsoft.com>
Mon, 9 Jan 2017 19:36:28 +0000 (11:36 -0800)
src/jit/liveness.cpp

index 79f3b8d..1f60e3b 100644 (file)
@@ -1074,8 +1074,8 @@ class LiveVarAnalysis
 {
     Compiler* m_compiler;
 
-    bool      m_changed;
-    bool      m_hasPossibleBackEdge;
+    bool m_changed;
+    bool m_hasPossibleBackEdge;
 
     bool      m_heapLiveIn;
     bool      m_heapLiveOut;
@@ -1083,11 +1083,17 @@ class LiveVarAnalysis
     VARSET_TP m_liveOut;
 
     LiveVarAnalysis(Compiler* compiler)
-        : m_compiler(compiler), m_changed(false), m_hasPossibleBackEdge(false), m_heapLiveIn(false), m_heapLiveOut(false), m_liveIn(VarSetOps::MakeEmpty(compiler)), m_liveOut(VarSetOps::MakeEmpty(compiler))
+        : m_compiler(compiler)
+        , m_changed(false)
+        , m_hasPossibleBackEdge(false)
+        , m_heapLiveIn(false)
+        , m_heapLiveOut(false)
+        , m_liveIn(VarSetOps::MakeEmpty(compiler))
+        , m_liveOut(VarSetOps::MakeEmpty(compiler))
     {
     }
 
-    void PerBlock(BasicBlock* block, bool updateInternalOnly, bool keepAliveThis)
+    void PerBlockAnalysis(BasicBlock* block, bool updateInternalOnly, bool keepAliveThis)
     {
         /* Compute the 'liveOut' set */
         VarSetOps::ClearD(m_compiler, m_liveOut);
@@ -1149,7 +1155,8 @@ class LiveVarAnalysis
 
         /* Has there been any change in either live set? */
 
-        if (!VarSetOps::Equal(m_compiler, block->bbLiveIn, m_liveIn) || !VarSetOps::Equal(m_compiler, block->bbLiveOut, m_liveOut))
+        if (!VarSetOps::Equal(m_compiler, block->bbLiveIn, m_liveIn) ||
+            !VarSetOps::Equal(m_compiler, block->bbLiveOut, m_liveOut))
         {
             if (updateInternalOnly)
             {
@@ -1157,8 +1164,10 @@ class LiveVarAnalysis
 
                 noway_assert(block->bbFlags & BBF_INTERNAL);
 
-                if (!VarSetOps::Equal(m_compiler, VarSetOps::Intersection(m_compiler, block->bbLiveIn, m_liveIn), m_liveIn) ||
-                    !VarSetOps::Equal(m_compiler, VarSetOps::Intersection(m_compiler, block->bbLiveOut, m_liveOut), m_liveOut))
+                if (!VarSetOps::Equal(m_compiler, VarSetOps::Intersection(m_compiler, block->bbLiveIn, m_liveIn),
+                                      m_liveIn) ||
+                    !VarSetOps::Equal(m_compiler, VarSetOps::Intersection(m_compiler, block->bbLiveOut, m_liveOut),
+                                      m_liveOut))
                 {
 #ifdef DEBUG
                     if (m_compiler->verbose)
@@ -1194,7 +1203,8 @@ class LiveVarAnalysis
 
     void Run(bool updateInternalOnly)
     {
-        const bool keepAliveThis = m_compiler->lvaKeepAliveAndReportThis() && m_compiler->lvaTable[m_compiler->info.compThisArg].lvTracked;
+        const bool keepAliveThis =
+            m_compiler->lvaKeepAliveAndReportThis() && m_compiler->lvaTable[m_compiler->info.compThisArg].lvTracked;
 
         /* Live Variable Analysis - Backward dataflow */
         do
@@ -1231,7 +1241,7 @@ class LiveVarAnalysis
                     }
                 }
 
-                PerBlock(block, updateInternalOnly, keepAliveThis);
+                PerBlockAnalysis(block, updateInternalOnly, keepAliveThis);
             }
             // if there is no way we could have processed a block without seeing all of its predecessors
             // then there is no need to iterate