Updated to be a bit more const friendly. Also switched the linkage to be static since this function is file-local. No functional changes intended.

llvm-svn: 206982

1 files changed, 3 insertions, 3 deletions

diff --git a/clang/lib/Analysis/ThreadSafetyCommon.cpp b/clang/lib/Analysis/ThreadSafetyCommon.cpp
index 391680918e3..252fd97496b 100644
--- a/clang/lib/Analysis/ThreadSafetyCommon.cpp
+++ b/clang/lib/Analysis/ThreadSafetyCommon.cpp
@@ -97,9 +97,9 @@ void simplifyIncompleteArg(Variable *V, til::Phi *Ph) {
 
 
 // Return true if E is a variable that points to an incomplete Phi node.
-inline bool isIncompleteVar(SExpr *E) {
-  if (Variable *V = dyn_cast<Variable>(E)) {
-    if (Phi *Ph = dyn_cast<Phi>(V->definition()))
+static bool isIncompleteVar(const SExpr *E) {
+  if (const auto *V = dyn_cast<Variable>(E)) {
+    if (const auto *Ph = dyn_cast<Phi>(V->definition()))
       return Ph->status() == Phi::PH_Incomplete;
   }
   return false;