From bf4e0959ad4929d245107ceb91e1738a10237b0a Mon Sep 17 00:00:00 2001 From: Aaron Ballman Date: Thu, 6 Mar 2014 19:37:24 +0000 Subject: [PATCH] Based on usage, the NamedDecl is always set when constructing a CallingContext, but none of the other optional parameters are. Removing the optional parameters, and making the NamedDecl required. No functional changes intended. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@203149 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/ThreadSafety.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/Analysis/ThreadSafety.cpp b/lib/Analysis/ThreadSafety.cpp index c21254708d..6e9845db34 100644 --- a/lib/Analysis/ThreadSafety.cpp +++ b/lib/Analysis/ThreadSafety.cpp @@ -172,12 +172,9 @@ private: const Expr* const* FunArgs; // Function arguments CallingContext* PrevCtx; // The previous context; or 0 if none. - CallingContext(const NamedDecl *D = 0, const Expr *S = 0, - unsigned N = 0, const Expr* const *A = 0, - CallingContext *P = 0) - : AttrDecl(D), SelfArg(S), SelfArrow(false), - NumArgs(N), FunArgs(A), PrevCtx(P) - { } + CallingContext(const NamedDecl *D) + : AttrDecl(D), SelfArg(0), SelfArrow(false), NumArgs(0), FunArgs(0), + PrevCtx(0) {} }; typedef SmallVector NodeVector; -- 2.40.0