From: Pat Gavlin <pagavlin@microsoft.com>
Date: Mon, 9 Jan 2017 19:36:28 +0000 (-0800)
Subject: Format code and improve a function name.
X-Git-Tag: accepted/tizen/base/20180629.140029~2653^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=27361397bfe65aed784c2bd7031f0f28d27616da;p=platform%2Fupstream%2Fcoreclr.git

Format code and improve a function name.
---

diff --git a/src/jit/liveness.cpp b/src/jit/liveness.cpp
index 79f3b8d..1f60e3b 100644
--- a/src/jit/liveness.cpp
+++ b/src/jit/liveness.cpp
@@ -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