diff options
| author | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-20 13:23:58 +0000 |
|---|---|---|
| committer | Angel Garcia Gomez <angelgarcia@google.com> | 2015-10-20 13:23:58 +0000 |
| commit | d162035b9b1614af079d6ba056902eed2bc9bad6 (patch) | |
| tree | af1f98e5066ccac7ba2cdab335b1432805baca7b | |
| parent | e83bf34da93785210fe557b00803b6b91caa48b5 (diff) | |
Roll-back r250822.
Summary: It breaks the build for the ASTMatchers
Subscribers: klimek, cfe-commits
Differential Revision: http://reviews.llvm.org/D13893
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@250827 91177308-0d34-0410-b5e6-96231b3b80d8
150 files changed, 232 insertions, 222 deletions
diff --git a/include/clang/AST/ASTConsumer.h b/include/clang/AST/ASTConsumer.h index 02669e9bd1..b2730e4622 100644 --- a/include/clang/AST/ASTConsumer.h +++ b/include/clang/AST/ASTConsumer.h @@ -43,7 +43,7 @@ class ASTConsumer { public: ASTConsumer() : SemaConsumer(false) { } - virtual ~ASTConsumer() = default; + virtual ~ASTConsumer() {} /// Initialize - This is called to initialize the consumer, providing the /// ASTContext. diff --git a/include/clang/AST/CanonicalType.h b/include/clang/AST/CanonicalType.h index 6e0f88ee7d..b25800bfed 100644 --- a/include/clang/AST/CanonicalType.h +++ b/include/clang/AST/CanonicalType.h @@ -333,7 +333,7 @@ template<typename T> class CanProxy : public CanProxyAdaptor<T> { public: /// \brief Build a NULL proxy. - CanProxy() = default; + CanProxy() { } /// \brief Build a proxy to the given canonical type. CanProxy(CanQual<T> Stored) { this->Stored = Stored; } @@ -389,7 +389,7 @@ struct CanTypeIterator CanQualType, typename std::iterator_traits<InputIterator>::difference_type, CanProxy<Type>, CanQualType> { - CanTypeIterator() = default; + CanTypeIterator() {} explicit CanTypeIterator(InputIterator Iter) : CanTypeIterator::iterator_adaptor_base(std::move(Iter)) {} diff --git a/include/clang/AST/DeclFriend.h b/include/clang/AST/DeclFriend.h index 5bd6da8797..12b93b408a 100644 --- a/include/clang/AST/DeclFriend.h +++ b/include/clang/AST/DeclFriend.h @@ -180,7 +180,7 @@ class CXXRecordDecl::friend_iterator { friend class CXXRecordDecl; explicit friend_iterator(FriendDecl *Ptr) : Ptr(Ptr) {} public: - friend_iterator() = default; + friend_iterator() {} typedef FriendDecl *value_type; typedef FriendDecl *reference; diff --git a/include/clang/AST/DeclTemplate.h b/include/clang/AST/DeclTemplate.h index 3c6ef81442..bd38ac8823 100644 --- a/include/clang/AST/DeclTemplate.h +++ b/include/clang/AST/DeclTemplate.h @@ -664,7 +664,7 @@ protected: typename std::iterator_traits<typename llvm::FoldingSetVector< EntryType>::iterator>::iterator_category, DeclType *, ptrdiff_t, DeclType *, DeclType *> { - SpecIterator() = default; + SpecIterator() {} explicit SpecIterator( typename llvm::FoldingSetVector<EntryType>::iterator SetIter) : SpecIterator::iterator_adaptor_base(std::move(SetIter)) {} diff --git a/include/clang/AST/Expr.h b/include/clang/AST/Expr.h index 2e5de8768f..cc1fc0ae18 100644 --- a/include/clang/AST/Expr.h +++ b/include/clang/AST/Expr.h @@ -337,7 +337,7 @@ public: {} public: - Classification() = default; + Classification() {} Kinds getKind() const { return static_cast<Kinds>(Kind); } ModifiableType getModifiable() const { @@ -4076,7 +4076,7 @@ public: friend class DesignatedInitExpr; public: - Designator() = default; + Designator() {} /// @brief Initializes a field designator. Designator(const IdentifierInfo *FieldName, SourceLocation DotLoc, diff --git a/include/clang/AST/ExprCXX.h b/include/clang/AST/ExprCXX.h index 45d852ae0e..e8493f1933 100644 --- a/include/clang/AST/ExprCXX.h +++ b/include/clang/AST/ExprCXX.h @@ -2278,7 +2278,7 @@ public: : Expr(ArrayTypeTraitExprClass, Empty), ATT(0), Value(false), QueriedType() { } - virtual ~ArrayTypeTraitExpr() = default; + virtual ~ArrayTypeTraitExpr() { } SourceLocation getLocStart() const LLVM_READONLY { return Loc; } SourceLocation getLocEnd() const LLVM_READONLY { return RParen; } diff --git a/include/clang/AST/Mangle.h b/include/clang/AST/Mangle.h index 29d79592e5..7b725b65ca 100644 --- a/include/clang/AST/Mangle.h +++ b/include/clang/AST/Mangle.h @@ -64,7 +64,7 @@ public: ManglerKind Kind) : Context(Context), Diags(Diags), Kind(Kind) {} - virtual ~MangleContext() = default; + virtual ~MangleContext() { } ASTContext &getASTContext() const { return Context; } diff --git a/include/clang/AST/MangleNumberingContext.h b/include/clang/AST/MangleNumberingContext.h index fd15e1a98b..7a818557fd 100644 --- a/include/clang/AST/MangleNumberingContext.h +++ b/include/clang/AST/MangleNumberingContext.h @@ -32,7 +32,7 @@ class VarDecl; /// literals within a particular context. class MangleNumberingContext : public RefCountedBase<MangleNumberingContext> { public: - virtual ~MangleNumberingContext() = default; + virtual ~MangleNumberingContext() {} /// \brief Retrieve the mangling number of a new lambda expression with the /// given call operator within this context. diff --git a/include/clang/AST/Redeclarable.h b/include/clang/AST/Redeclarable.h index fd397bec0a..92046d582b 100644 --- a/include/clang/AST/Redeclarable.h +++ b/include/clang/AST/Redeclarable.h @@ -248,7 +248,7 @@ Decl *getPrimaryMergedDecl(Decl *D); template<typename decl_type> class Mergeable { public: - Mergeable() = default; + Mergeable() {} /// \brief Return the first declaration of this declaration or itself if this /// is the only declaration. diff --git a/include/clang/AST/StmtIterator.h b/include/clang/AST/StmtIterator.h index ced6ac8316..81f8ad4344 100644 --- a/include/clang/AST/StmtIterator.h +++ b/include/clang/AST/StmtIterator.h @@ -81,7 +81,7 @@ class StmtIteratorImpl : public StmtIteratorBase, protected: StmtIteratorImpl(const StmtIteratorBase& RHS) : StmtIteratorBase(RHS) {} public: - StmtIteratorImpl() = default; + StmtIteratorImpl() {} StmtIteratorImpl(Stmt **s) : StmtIteratorBase(s) {} StmtIteratorImpl(Decl **dgi, Decl **dge) : StmtIteratorBase(dgi, dge) {} StmtIteratorImpl(const VariableArrayType *t) : StmtIteratorBase(t) {} diff --git a/include/clang/AST/UnresolvedSet.h b/include/clang/AST/UnresolvedSet.h index af2f30fe34..26ee1cf71c 100644 --- a/include/clang/AST/UnresolvedSet.h +++ b/include/clang/AST/UnresolvedSet.h @@ -59,7 +59,7 @@ class UnresolvedSetImpl { // UnresolvedSet. private: template <unsigned N> friend class UnresolvedSet; - UnresolvedSetImpl() = default; + UnresolvedSetImpl() {} UnresolvedSetImpl(const UnresolvedSetImpl &) {} public: diff --git a/include/clang/AST/VTableBuilder.h b/include/clang/AST/VTableBuilder.h index 385a817303..481fd11d6a 100644 --- a/include/clang/AST/VTableBuilder.h +++ b/include/clang/AST/VTableBuilder.h @@ -298,7 +298,7 @@ public: bool isMicrosoft() const { return IsMicrosoftABI; } - virtual ~VTableContextBase() = default; + virtual ~VTableContextBase() {} protected: typedef llvm::DenseMap<const CXXMethodDecl *, ThunkInfoVectorTy> ThunksMapTy; diff --git a/include/clang/ASTMatchers/ASTMatchersInternal.h b/include/clang/ASTMatchers/ASTMatchersInternal.h index e703319f61..8e1abc6f77 100644 --- a/include/clang/ASTMatchers/ASTMatchersInternal.h +++ b/include/clang/ASTMatchers/ASTMatchersInternal.h @@ -131,7 +131,7 @@ public: /// BoundNodesTree. class Visitor { public: - virtual ~Visitor() = default; + virtual ~Visitor() {} /// \brief Called multiple times during a single call to VisitMatches(...). /// @@ -209,7 +209,7 @@ public: template <typename T> class MatcherInterface : public DynMatcherInterface { public: - ~MatcherInterface() override = default; + ~MatcherInterface() override {} /// \brief Returns true if 'Node' can be matched. /// @@ -798,7 +798,7 @@ public: AMM_ParentOnly }; - virtual ~ASTMatchFinder() = default; + virtual ~ASTMatchFinder() {} /// \brief Returns true if the given class is directly or indirectly derived /// from a base type matching \c base. @@ -1376,7 +1376,7 @@ class VariadicDynCastAllOfMatcher BindableMatcher<SourceT>, Matcher<TargetT>, makeDynCastAllOfComposite<SourceT, TargetT> > { public: - VariadicDynCastAllOfMatcher() = default; + VariadicDynCastAllOfMatcher() {} }; /// \brief A \c VariadicAllOfMatcher<T> object is a variadic functor that takes @@ -1394,7 +1394,7 @@ class VariadicAllOfMatcher : public llvm::VariadicFunction< BindableMatcher<T>, Matcher<T>, makeAllOfComposite<T> > { public: - VariadicAllOfMatcher() = default; + VariadicAllOfMatcher() {} }; /// \brief Matches nodes of type \c TLoc for which the inner @@ -1515,7 +1515,7 @@ public: struct Func : public llvm::VariadicFunction<Self, Matcher<InnerTBase>, &Self::create> { - Func() = default; + Func() {} }; private: diff --git a/include/clang/Analysis/Analyses/FormatString.h b/include/clang/Analysis/Analyses/FormatString.h index fbcc8a5fbf..4471311a33 100644 --- a/include/clang/Analysis/Analyses/FormatString.h +++ b/include/clang/Analysis/Analyses/FormatString.h @@ -606,7 +606,7 @@ enum PositionContext { FieldWidthPos = 0, PrecisionPos = 1 }; class FormatStringHandler { public: - FormatStringHandler() = default; + FormatStringHandler() {} virtual ~FormatStringHandler(); virtual void HandleNullChar(const char *nullCharacter) {} diff --git a/include/clang/Analysis/Analyses/LiveVariables.h b/include/clang/Analysis/Analyses/LiveVariables.h index c6c0ece13b..e17f73a61f 100644 --- a/include/clang/Analysis/Analyses/LiveVariables.h +++ b/include/clang/Analysis/Analyses/LiveVariables.h @@ -53,7 +53,7 @@ public: class Observer { virtual void anchor(); public: - virtual ~Observer() = default; + virtual ~Observer() {} /// A callback invoked right before invoking the /// liveness transfer function on the given statement. diff --git a/include/clang/Analysis/Analyses/ReachableCode.h b/include/clang/Analysis/Analyses/ReachableCode.h index a388cc9fec..4c523bfc8b 100644 --- a/include/clang/Analysis/Analyses/ReachableCode.h +++ b/include/clang/Analysis/Analyses/ReachableCode.h @@ -48,7 +48,7 @@ enum UnreachableKind { class Callback { virtual void anchor(); public: - virtual ~Callback() = default; + virtual ~Callback() {} virtual void HandleUnreachable(UnreachableKind UK, SourceLocation L, SourceRange ConditionVal, diff --git a/include/clang/Analysis/Analyses/ThreadSafetyTraverse.h b/include/clang/Analysis/Analyses/ThreadSafetyTraverse.h index 5bf0ea496b..705fe910d0 100644 --- a/include/clang/Analysis/Analyses/ThreadSafetyTraverse.h +++ b/include/clang/Analysis/Analyses/ThreadSafetyTraverse.h @@ -176,7 +176,7 @@ template <class Self> class VisitReducer : public Traversal<Self, VisitReducerBase>, public VisitReducerBase { public: - VisitReducer() = default; + VisitReducer() {} public: R_SExpr reduceNull() { return true; } diff --git a/include/clang/Analysis/Analyses/UninitializedValues.h b/include/clang/Analysis/Analyses/UninitializedValues.h index 18ed02dd2e..53ff20c235 100644 --- a/include/clang/Analysis/Analyses/UninitializedValues.h +++ b/include/clang/Analysis/Analyses/UninitializedValues.h @@ -99,7 +99,7 @@ public: class UninitVariablesHandler { public: - UninitVariablesHandler() = default; + UninitVariablesHandler() {} virtual ~UninitVariablesHandler(); /// Called when the uninitialized variable is used at the given expression. diff --git a/include/clang/Analysis/AnalysisContext.h b/include/clang/Analysis/AnalysisContext.h index 124ec3dda5..931190e43a 100644 --- a/include/clang/Analysis/AnalysisContext.h +++ b/include/clang/Analysis/AnalysisContext.h @@ -44,7 +44,7 @@ namespace idx { class TranslationUnit; } /// to AnalysisDeclContext. class ManagedAnalysis { protected: - ManagedAnalysis() = default; + ManagedAnalysis() {} public: virtual ~ManagedAnalysis(); @@ -289,7 +289,7 @@ class StackFrameContext : public LocationContext { Block(blk), Index(idx) {} public: - ~StackFrameContext() override = default; + ~StackFrameContext() override {} const Stmt *getCallSite() const { return CallSite; } @@ -324,7 +324,7 @@ class ScopeContext : public LocationContext { : LocationContext(Scope, ctx, parent), Enter(s) {} public: - ~ScopeContext() override = default; + ~ScopeContext() override {} void Profile(llvm::FoldingSetNodeID &ID) override; @@ -352,7 +352,7 @@ class BlockInvocationContext : public LocationContext { : LocationContext(Block, ctx, parent), BD(bd), ContextData(contextData) {} public: - ~BlockInvocationContext() override = default; + ~BlockInvocationContext() override {} const BlockDecl *getBlockDecl() const { return BD; } diff --git a/include/clang/Analysis/CFG.h b/include/clang/Analysis/CFG.h index 58e26604dc..293990c88e 100644 --- a/include/clang/Analysis/CFG.h +++ b/include/clang/Analysis/CFG.h @@ -704,11 +704,11 @@ public: /// operator error is found when building the CFG. class CFGCallback { public: - CFGCallback() = default; + CFGCallback() {} virtual void compareAlwaysTrue(const BinaryOperator *B, bool isAlwaysTrue) {} virtual void compareBitwiseEquality(const BinaryOperator *B, bool isAlwaysTrue) {} - virtual ~CFGCallback() = default; + virtual ~CFGCallback() {} }; /// CFG - Represents a source-level, intra-procedural CFG that represents the diff --git a/include/clang/Basic/FileSystemStatCache.h b/include/clang/Basic/FileSystemStatCache.h index 430851df67..cad9189348 100644 --- a/include/clang/Basic/FileSystemStatCache.h +++ b/include/clang/Basic/FileSystemStatCache.h @@ -51,7 +51,7 @@ protected: std::unique_ptr<FileSystemStatCache> NextStatCache; public: - virtual ~FileSystemStatCache() = default; + virtual ~FileSystemStatCache() {} enum LookupResult { CacheExists, ///< We know the file exists and its cached stat data. diff --git a/include/clang/Basic/IdentifierTable.h b/include/clang/Basic/IdentifierTable.h index b0812ccd90..d672314f56 100644 --- a/include/clang/Basic/IdentifierTable.h +++ b/include/clang/Basic/IdentifierTable.h @@ -385,7 +385,7 @@ private: void operator=(const IdentifierIterator &) = delete; protected: - IdentifierIterator() = default; + IdentifierIterator() { } public: virtual ~IdentifierIterator(); diff --git a/include/clang/Driver/Job.h b/include/clang/Driver/Job.h index d1fd6e3465..263356f396 100644 --- a/include/clang/Driver/Job.h +++ b/include/clang/Driver/Job.h @@ -88,7 +88,7 @@ public: // FIXME: This really shouldn't be copyable, but is currently copied in some // error handling in Driver::generateCompilationDiagnostics. Command(const Command &) = default; - virtual ~Command() = default; + virtual ~Command() {} virtual void Print(llvm::raw_ostream &OS, const char *Terminator, bool Quote, CrashReportInfo *CrashInfo = nullptr) const; diff --git a/include/clang/Edit/EditsReceiver.h b/include/clang/Edit/EditsReceiver.h index 1b63d0ee86..600ac28ea9 100644 --- a/include/clang/Edit/EditsReceiver.h +++ b/include/clang/Edit/EditsReceiver.h @@ -20,7 +20,7 @@ namespace edit { class EditsReceiver { public: - virtual ~EditsReceiver() = default; + virtual ~EditsReceiver() { } virtual void insert(SourceLocation loc, StringRef text) = 0; virtual void replace(CharSourceRange range, StringRef text) = 0; diff --git a/include/clang/Frontend/SerializedDiagnosticReader.h b/include/clang/Frontend/SerializedDiagnosticReader.h index 5c0a06f4c1..3db362bf34 100644 --- a/include/clang/Frontend/SerializedDiagnosticReader.h +++ b/include/clang/Frontend/SerializedDiagnosticReader.h @@ -58,8 +58,8 @@ struct Location { /// the various constructs that are found in serialized diagnostics. class SerializedDiagnosticReader { public: - SerializedDiagnosticReader() = default; - virtual ~SerializedDiagnosticReader() = default; + SerializedDiagnosticReader() {} + virtual ~SerializedDiagnosticReader() {} /// \brief Read the diagnostics in \c File std::error_code readDiagnostics(StringRef File); diff --git a/include/clang/Frontend/VerifyDiagnosticConsumer.h b/include/clang/Frontend/VerifyDiagnosticConsumer.h index 493dac3b11..475f07f9dc 100644 --- a/include/clang/Frontend/VerifyDiagnosticConsumer.h +++ b/include/clang/Frontend/VerifyDiagnosticConsumer.h @@ -161,7 +161,7 @@ public: unsigned Min, Max; bool MatchAnyLine; - virtual ~Directive() = default; + virtual ~Directive() { } // Returns true if directive text is valid. // Otherwise returns false and populates E. diff --git a/include/clang/Lex/ModuleMap.h b/include/clang/Lex/ModuleMap.h index 3320843243..155943e545 100644 --- a/include/clang/Lex/ModuleMap.h +++ b/include/clang/Lex/ModuleMap.h @@ -40,7 +40,7 @@ class ModuleMapParser; /// reads module map files. class ModuleMapCallbacks { public: - virtual ~ModuleMapCallbacks() = default; + virtual ~ModuleMapCallbacks() {} /// \brief Called when a module map file has been read. /// diff --git a/include/clang/Lex/PTHLexer.h b/include/clang/Lex/PTHLexer.h index e2e2306c0e..904be792b2 100644 --- a/include/clang/Lex/PTHLexer.h +++ b/include/clang/Lex/PTHLexer.h @@ -64,7 +64,7 @@ protected: PTHLexer(Preprocessor& pp, FileID FID, const unsigned char *D, const unsigned char* ppcond, PTHManager &PM); public: - ~PTHLexer() override = default; + ~PTHLexer() override {} /// Lex - Return the next token. bool Lex(Token &Tok); diff --git a/include/clang/Lex/PreprocessorLexer.h b/include/clang/Lex/PreprocessorLexer.h index dfc7768f62..6d6cf05a96 100644 --- a/include/clang/Lex/PreprocessorLexer.h +++ b/include/clang/Lex/PreprocessorLexer.h @@ -81,7 +81,7 @@ protected: ParsingFilename(false), LexingRawMode(false) {} - virtual ~PreprocessorLexer() = default; + virtual ~PreprocessorLexer() {} virtual void IndirectLex(Token& Result) = 0; diff --git a/include/clang/Sema/Sema.h b/include/clang/Sema/Sema.h index 09d6a4b850..cce5405a1e 100644 --- a/include/clang/Sema/Sema.h +++ b/include/clang/Sema/Sema.h @@ -1313,7 +1313,7 @@ public: TypeDiagnoser(bool Suppressed = false) : Suppressed(Suppressed) { } virtual void diagnose(Sema &S, SourceLocation Loc, QualType T) = 0; - virtual ~TypeDiagnoser() = default; + virtual ~TypeDiagnoser() {} }; static int getPrintable(int I) { return I; } @@ -2297,7 +2297,7 @@ public: virtual SemaDiagnosticBuilder diagnoseConversion( Sema &S, SourceLocation Loc, QualType T, QualType ConvTy) = 0; - virtual ~ContextualImplicitConverter() = default; + virtual ~ContextualImplicitConverter() {} }; class ICEConvertDiagnoser : public ContextualImplicitConverter { @@ -8597,7 +8597,7 @@ public: virtual void diagnoseNotICE(Sema &S, SourceLocation Loc, SourceRange SR) =0; virtual void diagnoseFold(Sema &S, SourceLocation Loc, SourceRange SR); - virtual ~VerifyICEDiagnoser() = default; + virtual ~VerifyICEDiagnoser() { } }; /// VerifyIntegerConstantExpression - Verifies that an expression is an ICE, diff --git a/include/clang/Sema/TypoCorrection.h b/include/clang/Sema/TypoCorrection.h index 6c8a2852df..958aab0fce 100644 --- a/include/clang/Sema/TypoCorrection.h +++ b/include/clang/Sema/TypoCorrection.h @@ -255,7 +255,7 @@ public: IsObjCIvarLookup(false), IsAddressOfOperand(false), Typo(Typo), TypoNNS(TypoNNS) {} - virtual ~CorrectionCandidateCallback() = default; + virtual ~CorrectionCandidateCallback() {} /// \brief Simple predicate used by the default RankCandidate to /// determine whether to return an edit distance of 0 or InvalidDistance. diff --git a/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h b/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h index b8133bfd12..57c73fd6ec 100644 --- a/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h +++ b/include/clang/StaticAnalyzer/Core/BugReporter/BugReporter.h @@ -57,7 +57,7 @@ public: class NodeResolver { virtual void anchor(); public: - virtual ~NodeResolver() = default; + virtual ~NodeResolver() {} virtual const ExplodedNode* getOriginalNode(const ExplodedNode *N) = 0; }; @@ -529,7 +529,7 @@ class BugReporterContext { public: BugReporterContext(GRBugReporter& br) : BR(br) {} - virtual ~BugReporterContext() = default; + virtual ~BugReporterContext() {} GRBugReporter& getBugReporter() { return BR; } diff --git a/include/clang/StaticAnalyzer/Core/BugReporter/BugType.h b/include/clang/StaticAnalyzer/Core/BugReporter/BugType.h index 60f91de6df..16226e94df 100644 --- a/include/clang/StaticAnalyzer/Core/BugReporter/BugType.h +++ b/include/clang/StaticAnalyzer/Core/BugReporter/BugType.h @@ -42,7 +42,7 @@ public: BugType(const CheckerBase *checker, StringRef name, StringRef cat) : Check(checker->getCheckName()), Name(name), Category(cat), SuppressonSink(false) {} - virtual ~BugType() = default; + virtual ~BugType() {} // FIXME: Should these be made strings as well? StringRef getName() const { return Name; } diff --git a/include/clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h b/include/clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h index ad083f7122..35421f9455 100644 --- a/include/clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h +++ b/include/clang/StaticAnalyzer/Core/BugReporter/PathDiagnostic.h @@ -481,7 +481,7 @@ private: public: StackHintGeneratorForSymbol(SymbolRef S, StringRef M) : Sym(S), Msg(M) {} - ~StackHintGeneratorForSymbol() override = default; + ~StackHintGeneratorForSymbol() override {} /// \brief Search the call expression for the symbol Sym and dispatch the /// 'getMessageForX()' methods to construct a specific message. diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h b/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h index 7fe8e3d3db..cf888b5040 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h @@ -168,7 +168,7 @@ protected: RegionAndSymbolInvalidationTraits *ETraits) const {} public: - virtual ~CallEvent() = default; + virtual ~CallEvent() {} /// \brief Returns the kind of call this is. virtual Kind getKind() const = 0; diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/CoreEngine.h b/include/clang/StaticAnalyzer/Core/PathSensitive/CoreEngine.h index 368fdee4be..d5822e2244 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/CoreEngine.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/CoreEngine.h @@ -254,7 +254,7 @@ public: assert(hasNoSinksInFrontier()); } - virtual ~NodeBuilder() = default; + virtual ~NodeBuilder() {} /// \brief Generates a node in the ExplodedGraph. ExplodedNode *generateNode(const ProgramPoint &PP, diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h b/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h index 9ae9aaf6de..a68d3410a8 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/SValBuilder.h @@ -68,7 +68,7 @@ public: ArrayIndexTy(context.IntTy), ArrayIndexWidth(context.getTypeSize(ArrayIndexTy)) {} - virtual ~SValBuilder() = default; + virtual ~SValBuilder() {} bool haveSameType(const SymExpr *Sym1, const SymExpr *Sym2) { return haveSameType(Sym1->getType(), Sym2->getType()); diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/Store.h b/include/clang/StaticAnalyzer/Core/PathSensitive/Store.h index b29e5f7047..a03b6306a0 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/Store.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/Store.h @@ -49,7 +49,7 @@ protected: StoreManager(ProgramStateManager &stateMgr); public: - virtual ~StoreManager() = default; + virtual ~StoreManager() {} /// Return the value bound to specified location in a given state. /// \param[in] store The analysis state. diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/SubEngine.h b/include/clang/StaticAnalyzer/Core/PathSensitive/SubEngine.h index fa3f200ba8..741ba0e2f2 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/SubEngine.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/SubEngine.h @@ -43,7 +43,7 @@ class MemRegion; class SubEngine { virtual void anchor(); public: - virtual ~SubEngine() = default; + virtual ~SubEngine() {} virtual ProgramStateRef getInitialState(const LocationContext *InitLoc) = 0; diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h b/include/clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h index cfb7edec8e..05de02db59 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h @@ -57,7 +57,7 @@ protected: SymExpr(Kind k) : K(k) {} public: - virtual ~SymExpr() = default; + virtual ~SymExpr() {} Kind getKind() const { return K; } @@ -109,7 +109,7 @@ protected: SymbolData(Kind k, SymbolID sym) : SymExpr(k), Sym(sym) {} public: - ~SymbolData() override = default; + ~SymbolData() override {} SymbolID getSymbolID() const { return Sym; } diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/TaintManager.h b/include/clang/StaticAnalyzer/Core/PathSensitive/TaintManager.h index d46efd4ef0..d39b5017d3 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/TaintManager.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/TaintManager.h @@ -37,7 +37,7 @@ template<> struct ProgramStateTrait<TaintMap> class TaintManager { - TaintManager() = default; + TaintManager() {} }; } diff --git a/include/clang/StaticAnalyzer/Core/PathSensitive/WorkList.h b/include/clang/StaticAnalyzer/Core/PathSensitive/WorkList.h index 24d4503f37..4f1a60e675 100644 --- a/include/clang/StaticAnalyzer/Core/PathSensitive/WorkList.h +++ b/include/clang/StaticAnalyzer/Core/PathSensitive/WorkList.h @@ -82,7 +82,7 @@ public: class Visitor { public: - Visitor() = default; + Visitor() {} virtual ~Visitor(); virtual bool visit(const WorkListUnit &U) = 0; }; diff --git a/include/clang/Tooling/FileMatchTrie.h b/include/clang/Tooling/FileMatchTrie.h index d7a6a379c2..745c164506 100644 --- a/include/clang/Tooling/FileMatchTrie.h +++ b/include/clang/Tooling/FileMatchTrie.h @@ -25,7 +25,7 @@ namespace clang { namespace tooling { struct PathComparator { - virtual ~PathComparator() = default; + virtual ~PathComparator() {} virtual bool equivalent(StringRef FileA, StringRef FileB) const = 0; }; class FileMatchTrieNode; diff --git a/include/clang/Tooling/Tooling.h b/include/clang/Tooling/Tooling.h index ada1f526b5..b7a9b25acd 100644 --- a/include/clang/Tooling/Tooling.h +++ b/include/clang/Tooling/Tooling.h @@ -110,7 +110,7 @@ std::unique_ptr<FrontendActionFactory> newFrontendActionFactory(); /// newFrontendActionFactory. class SourceFileCallbacks { public: - virtual ~SourceFileCallbacks() = default; + virtual ~SourceFileCallbacks() {} /// \brief Called before a source file is processed by a FrontEndAction. /// \see clang::FrontendAction::BeginSourceFileAction diff --git a/lib/ARCMigrate/ARCMT.cpp b/lib/ARCMigrate/ARCMT.cpp index c9239ded2b..7eb01fb93d 100644 --- a/lib/ARCMigrate/ARCMT.cpp +++ b/lib/ARCMigrate/ARCMT.cpp @@ -498,7 +498,7 @@ public: } // end anonymous namespace. /// \brief Anchor for VTable. -MigrationProcess::RewriteListener::~RewriteListener() = default; +MigrationProcess::RewriteListener::~RewriteListener() { } MigrationProcess::MigrationProcess( const CompilerInvocation &CI, diff --git a/lib/ARCMigrate/TransformActions.cpp b/lib/ARCMigrate/TransformActions.cpp index e7eb821691..c628b54ed4 100644 --- a/lib/ARCMigrate/TransformActions.cpp +++ b/lib/ARCMigrate/TransformActions.cpp @@ -594,7 +594,7 @@ SourceLocation TransformActionsImpl::getLocForEndOfToken(SourceLocation loc, return PP.getLocForEndOfToken(loc); } -TransformActions::RewriteReceiver::~RewriteReceiver() = default; +TransformActions::RewriteReceiver::~RewriteReceiver() { } TransformActions::TransformActions(DiagnosticsEngine &diag, CapturedDiagList &capturedDiags, diff --git a/lib/ARCMigrate/Transforms.cpp b/lib/ARCMigrate/Transforms.cpp index 2bdaf19ebc..56d3af7233 100644 --- a/lib/ARCMigrate/Transforms.cpp +++ b/lib/ARCMigrate/Transforms.cpp @@ -27,7 +27,7 @@ using namespace clang; using namespace arcmt; using namespace trans; -ASTTraverser::~ASTTraverser() = default; +ASTTraverser::~ASTTraverser() { } bool MigrationPass::CFBridgingFunctionsDefined() { if (!EnableCFBridgeFns.hasValue()) diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp index 1f1e00a812..848158877a 100644 --- a/lib/AST/ASTContext.cpp +++ b/lib/AST/ASTContext.cpp @@ -7914,7 +7914,7 @@ QualType ASTContext::getCorrespondingUnsignedType(QualType T) const { } } -ASTMutationListener::~ASTMutationListener() = default; +ASTMutationListener::~ASTMutationListener() { } void ASTMutationListener::DeducedReturnType(const FunctionDecl *FD, QualType ReturnType) {} @@ -8502,7 +8502,7 @@ MangleContext *ASTContext::createMangleContext() { llvm_unreachable("Unsupported ABI"); } -CXXABI::~CXXABI() = default; +CXXABI::~CXXABI() {} size_t ASTContext::getSideTableAllocatedMemory() const { return ASTRecordLayouts.getMemorySize() + diff --git a/lib/AST/ASTImporter.cpp b/lib/AST/ASTImporter.cpp index fded9444a5..6816dd11d9 100644 --- a/lib/AST/ASTImporter.cpp +++ b/lib/AST/ASTImporter.cpp @@ -5344,7 +5344,7 @@ ASTImporter::ASTImporter(ASTContext &ToContext, FileManager &ToFileManager, = ToContext.getTranslationUnitDecl(); } -ASTImporter::~ASTImporter() = default; +ASTImporter::~ASTImporter() { } QualType ASTImporter::Import(QualType FromT) { if (FromT.isNull()) diff --git a/lib/AST/DeclBase.cpp b/lib/AST/DeclBase.cpp index ff92630a26..406dff6899 100644 --- a/lib/AST/DeclBase.cpp +++ b/lib/AST/DeclBase.cpp @@ -236,7 +236,7 @@ void PrettyStackTraceDecl::print(raw_ostream &OS) const { //===----------------------------------------------------------------------===// // Out-of-line virtual method providing a home for Decl. -Decl::~Decl() = default; +Decl::~Decl() { } void Decl::setDeclContext(DeclContext *DC) { DeclCtx = DC; @@ -836,7 +836,7 @@ bool DeclContext::classof(const Decl *D) { } } -DeclContext::~DeclContext() = default; +DeclContext::~DeclContext() { } /// \brief Find the parent context of this context that will be /// used for unqualified name lookup. diff --git a/lib/AST/ExternalASTSource.cpp b/lib/AST/ExternalASTSource.cpp index c42a8b4fcd..e3de8c5fef 100644 --- a/lib/AST/ExternalASTSource.cpp +++ b/lib/AST/ExternalASTSource.cpp @@ -21,7 +21,7 @@ using namespace clang; -ExternalASTSource::~ExternalASTSource() = default; +ExternalASTSource::~ExternalASTSource() { } llvm::Optional<ExternalASTSource::ASTSourceDescriptor> ExternalASTSource::getSourceDescriptor(unsigned ID) { diff --git a/lib/AST/StmtPrinter.cpp b/lib/AST/StmtPrinter.cpp index 21c2959f64..bf155f5680 100644 --- a/lib/AST/StmtPrinter.cpp +++ b/lib/AST/StmtPrinter.cpp @@ -2338,4 +2338,4 @@ void Stmt::printPretty(raw_ostream &OS, //===----------------------------------------------------------------------===// // Implement virtual destructor. -PrinterHelper::~PrinterHelper() = default; +PrinterHelper::~PrinterHelper() {} diff --git a/lib/AST/VTableBuilder.cpp b/lib/AST/VTableBuilder.cpp index 747a1bda36..6829da5bb1 100644 --- a/lib/AST/VTableBuilder.cpp +++ b/lib/AST/VTableBuilder.cpp @@ -2229,7 +2229,7 @@ VTableLayout::VTableLayout(uint64_t NumVTableComponents, }); } -VTableLayout::~VTableLayout() = default; +VTableLayout::~VTableLayout() { } ItaniumVTableContext::ItaniumVTableContext(ASTContext &Context) : VTableContextBase(/*MS=*/false) {} diff --git a/lib/ASTMatchers/ASTMatchFinder.cpp b/lib/ASTMatchers/ASTMatchFinder.cpp index f4cf5ef20f..8807b13c5e 100644 --- a/lib/ASTMatchers/ASTMatchFinder.cpp +++ b/lib/ASTMatchers/ASTMatchFinder.cpp @@ -902,13 +902,13 @@ MatchFinder::MatchResult::MatchResult(const BoundNodes &Nodes, : Nodes(Nodes), Context(Context), SourceManager(&Context->getSourceManager()) {} -MatchFinder::MatchCallback::~MatchCallback() = default; -MatchFinder::ParsingDoneTestCallback::~ParsingDoneTestCallback() = default; +MatchFinder::MatchCallback::~MatchCallback() {} +MatchFinder::ParsingDoneTestCallback::~ParsingDoneTestCallback() {} MatchFinder::MatchFinder(MatchFinderOptions Options) : Options(std::move(Options)), ParsingDone(nullptr) {} -MatchFinder::~MatchFinder() = default; +MatchFinder::~MatchFinder() {} void MatchFinder::addMatcher(const DeclarationMatcher &NodeMatch, MatchCallback *Action) { diff --git a/lib/ASTMatchers/Dynamic/Marshallers.h b/lib/ASTMatchers/Dynamic/Marshallers.h index 0d6fbf916c..64d6b7814a 100644 --- a/lib/ASTMatchers/Dynamic/Marshallers.h +++ b/lib/ASTMatchers/Dynamic/Marshallers.h @@ -103,7 +103,7 @@ public: /// arguments, and various other methods for type introspection. class MatcherDescriptor { public: - virtual ~MatcherDescriptor() = default; + virtual ~MatcherDescriptor() {} virtual VariantMatcher create(SourceRange NameRange, ArrayRef<ParserValue> Args, Diagnostics *Error) const = 0; @@ -491,7 +491,7 @@ public: OverloadedMatcherDescriptor(ArrayRef<MatcherDescriptor *> Callbacks) : Overloads(Callbacks.begin(), Callbacks.end()) {} - ~OverloadedMatcherDescriptor() override = default; + ~OverloadedMatcherDescriptor() override {} VariantMatcher create(SourceRange NameRange, ArrayRef<ParserValue> Args, diff --git a/lib/ASTMatchers/Dynamic/Parser.cpp b/lib/ASTMatchers/Dynamic/Parser.cpp index ca4a12716d..cf9dab6dc7 100644 --- a/lib/ASTMatchers/Dynamic/Parser.cpp +++ b/lib/ASTMatchers/Dynamic/Parser.cpp @@ -257,7 +257,7 @@ private: const char *CodeCompletionLocation; }; -Parser::Sema::~Sema() = default; +Parser::Sema::~Sema() {} std::vector<ArgKind> Parser::Sema::getAcceptedCompletionTypes( llvm::ArrayRef<std::pair<MatcherCtor, unsigned>> Context) { @@ -526,7 +526,7 @@ Parser::Parser(CodeTokenizer *Tokenizer, Sema *S, : Tokenizer(Tokenizer), S(S ? S : &*DefaultRegistrySema), NamedValues(NamedValues), Error(Error) {} -Parser::RegistrySema::~RegistrySema() = default; +Parser::RegistrySema::~RegistrySema() {} llvm::Optional<MatcherCtor> Parser::RegistrySema::lookupMatcherCtor(StringRef MatcherName) { diff --git a/lib/ASTMatchers/Dynamic/VariantValue.cpp b/lib/ASTMatchers/Dynamic/VariantValue.cpp index d73fb7b623..8f3c70c1a8 100644 --- a/lib/ASTMatchers/Dynamic/VariantValue.cpp +++ b/lib/ASTMatchers/Dynamic/VariantValue.cpp @@ -75,7 +75,7 @@ VariantMatcher::MatcherOps::constructVariadicOperator( return DynTypedMatcher::constructVariadic(Op, NodeKind, DynMatchers); } -VariantMatcher::Payload::~Payload() = default; +VariantMatcher::Payload::~Payload() {} class VariantMatcher::SinglePayload : public VariantMatcher::Payload { public: @@ -113,7 +113,7 @@ public: PolymorphicPayload(std::vector<DynTypedMatcher> MatchersIn) : Matchers(std::move(MatchersIn)) {} - ~PolymorphicPayload() override = default; + ~PolymorphicPayload() override {} llvm::Optional<DynTypedMatcher> getSingleMatcher() const override { if (Matchers.size() != 1) diff --git a/lib/Analysis/AnalysisDeclContext.cpp b/lib/Analysis/AnalysisDeclContext.cpp index a54710181d..d7fb7e95d7 100644 --- a/lib/Analysis/AnalysisDeclContext.cpp +++ b/lib/Analysis/AnalysisDeclContext.cpp @@ -551,7 +551,7 @@ ManagedAnalysis *&AnalysisDeclContext::getAnalysisImpl(const void *tag) { // Cleanup. //===----------------------------------------------------------------------===// -ManagedAnalysis::~ManagedAnalysis() = default; +ManagedAnalysis::~ManagedAnalysis() {} AnalysisDeclContext::~AnalysisDeclContext() { delete forcedBlkExprs; @@ -568,7 +568,7 @@ AnalysisDeclContextManager::~AnalysisDeclContextManager() { llvm::DeleteContainerSeconds(Contexts); } -LocationContext::~LocationContext() = default; +LocationContext::~LocationContext() {} LocationContextManager::~LocationContextManager() { clear(); diff --git a/lib/Analysis/CFG.cpp b/lib/Analysis/CFG.cpp index 63201c4595..6cb63f2b17 100644 --- a/lib/Analysis/CFG.cpp +++ b/lib/Analysis/CFG.cpp @@ -4010,7 +4010,7 @@ public: } } - ~StmtPrinterHelper() override = default; + ~StmtPrinterHelper() override {} const LangOptions &getLangOpts() const { return LangOpts; } void setBlockID(signed i) { currentBlock = i; } diff --git a/lib/Analysis/CodeInjector.cpp b/lib/Analysis/CodeInjector.cpp index 3ac94f9bc0..76bf364444 100644 --- a/lib/Analysis/CodeInjector.cpp +++ b/lib/Analysis/CodeInjector.cpp @@ -11,5 +11,5 @@ using namespace clang; -CodeInjector::CodeInjector() = default; -CodeInjector::~CodeInjector() = default; +CodeInjector::CodeInjector() {} +CodeInjector::~CodeInjector() {} diff --git a/lib/Analysis/Consumed.cpp b/lib/Analysis/Consumed.cpp index 11429fff45..9df23923b0 100644 --- a/lib/Analysis/Consumed.cpp +++ b/lib/Analysis/Consumed.cpp @@ -52,7 +52,7 @@ using namespace clang; using namespace consumed; // Key method definition -ConsumedWarningsHandlerBase::~ConsumedWarningsHandlerBase() = default; +ConsumedWarningsHandlerBase::~ConsumedWarningsHandlerBase() {} static SourceLocation getFirstStmtLoc(const CFGBlock *Block) { // Find the source location of the first statement in the block, if the block diff --git a/lib/Analysis/FormatString.cpp b/lib/Analysis/FormatString.cpp index f8bbafd549..0948bc0b08 100644 --- a/lib/Analysis/FormatString.cpp +++ b/lib/Analysis/FormatString.cpp @@ -26,7 +26,7 @@ using clang::analyze_format_string::ConversionSpecifier; using namespace clang; // Key function to FormatStringHandler. -FormatStringHandler::~FormatStringHandler() = default; +FormatStringHandler::~FormatStringHandler() {} //===----------------------------------------------------------------------===// // Functions for parsing format strings components in both printf and diff --git a/lib/Analysis/ProgramPoint.cpp b/lib/Analysis/ProgramPoint.cpp index 787e5bbca7..26b59bb71d 100644 --- a/lib/Analysis/ProgramPoint.cpp +++ b/lib/Analysis/ProgramPoint.cpp @@ -16,7 +16,7 @@ using namespace clang; -ProgramPointTag::~ProgramPointTag() = default; +ProgramPointTag::~ProgramPointTag() {} ProgramPoint ProgramPoint::getProgramPoint(const Stmt *S, ProgramPoint::Kind K, const LocationContext *LC, diff --git a/lib/Analysis/ThreadSafety.cpp b/lib/Analysis/ThreadSafety.cpp index 407b7f0e0d..bce5175ece 100644 --- a/lib/Analysis/ThreadSafety.cpp +++ b/lib/Analysis/ThreadSafety.cpp @@ -48,7 +48,7 @@ using namespace clang; using namespace threadSafety; // Key method definition -ThreadSafetyHandler::~ThreadSafetyHandler() = default; +ThreadSafetyHandler::~ThreadSafetyHandler() {} namespace { class TILPrinter : @@ -107,7 +107,7 @@ public: : CapabilityExpr(CE), LKind(LK), AcquireLoc(Loc), Asserted(Asrt), Declared(Declrd) {} - virtual ~FactEntry() = default; + virtual ~FactEntry() {} LockKind kind() const { return LKind; } SourceLocation loc() const { return AcquireLoc; } diff --git a/lib/Analysis/UninitializedValues.cpp b/lib/Analysis/UninitializedValues.cpp index 2ee94d5e8e..f2f791957a 100644 --- a/lib/Analysis/UninitializedValues.cpp +++ b/lib/Analysis/UninitializedValues.cpp @@ -837,7 +837,7 @@ struct PruneBlocksHandler : public UninitVariablesHandler { : hadUse(numBlocks, false), hadAnyUse(false), currentBlock(0) {} - ~PruneBlocksHandler() override = default; + ~PruneBlocksHandler() override {} /// Records if a CFGBlock had a potential use of an uninitialized variable. llvm::BitVector hadUse; @@ -922,4 +922,4 @@ void clang::runUninitializedVariablesAnalysis( } } -UninitVariablesHandler::~UninitVariablesHandler() = default; +UninitVariablesHandler::~UninitVariablesHandler() {} diff --git a/lib/Basic/Diagnostic.cpp b/lib/Basic/Diagnostic.cpp index 8c15f25b5b..0caa876a57 100644 --- a/lib/Basic/Diagnostic.cpp +++ b/lib/Basic/Diagnostic.cpp @@ -393,7 +393,7 @@ bool DiagnosticsEngine::EmitCurrentDiagnostic(bool Force) { } -DiagnosticConsumer::~DiagnosticConsumer() = default; +DiagnosticConsumer::~DiagnosticConsumer() {} void DiagnosticConsumer::HandleDiagnostic(DiagnosticsEngine::Level DiagLevel, const Diagnostic &Info) { @@ -981,7 +981,7 @@ bool DiagnosticConsumer::IncludeInDiagnosticCounts() const { return true; } void IgnoringDiagConsumer::anchor() { } -ForwardingDiagnosticConsumer::~ForwardingDiagnosticConsumer() = default; +ForwardingDiagnosticConsumer::~ForwardingDiagnosticConsumer() {} void ForwardingDiagnosticConsumer::HandleDiagnostic( DiagnosticsEngine::Level DiagLevel, diff --git a/lib/Basic/FileManager.cpp b/lib/Basic/FileManager.cpp index c04ebda130..21e6ea3ab4 100644 --- a/lib/Basic/FileManager.cpp +++ b/lib/Basic/FileManager.cpp @@ -605,5 +605,5 @@ void FileManager::PrintStats() const { } // Virtual destructors for abstract base classes that need live in Basic. -PCHContainerWriter::~PCHContainerWriter() = default; -PCHContainerReader::~PCHContainerReader() = default; +PCHContainerWriter::~PCHContainerWriter() {} +PCHContainerReader::~PCHContainerReader() {} diff --git a/lib/Basic/IdentifierTable.cpp b/lib/Basic/IdentifierTable.cpp index c60789d816..199815aa89 100644 --- a/lib/Basic/IdentifierTable.cpp +++ b/lib/Basic/IdentifierTable.cpp @@ -53,9 +53,9 @@ IdentifierInfo::IdentifierInfo() { // IdentifierTable Implementation //===----------------------------------------------------------------------===// -IdentifierIterator::~IdentifierIterator() = default; +IdentifierIterator::~IdentifierIterator() { } -IdentifierInfoLookup::~IdentifierInfoLookup() = default; +IdentifierInfoLookup::~IdentifierInfoLookup() {} namespace { /// \brief A simple identifier lookup iterator that represents an diff --git a/lib/Basic/SourceManager.cpp b/lib/Basic/SourceManager.cpp index 100f6e88f1..536ae7ff5c 100644 --- a/lib/Basic/SourceManager.cpp +++ b/lib/Basic/SourceManager.cpp @@ -2204,7 +2204,7 @@ LLVM_DUMP_METHOD void SourceManager::dump() const { } } -ExternalSLocEntrySource::~ExternalSLocEntrySource() = default; +ExternalSLocEntrySource::~ExternalSLocEntrySource() { } /// Return the amount of memory used by memory buffers, breaking down /// by heap-backed versus mmap'ed memory. diff --git a/lib/Basic/TargetInfo.cpp b/lib/Basic/TargetInfo.cpp index 6cde57d865..21368f8b11 100644 --- a/lib/Basic/TargetInfo.cpp +++ b/lib/Basic/TargetInfo.cpp @@ -100,7 +100,7 @@ TargetInfo::TargetInfo(const llvm::Triple &T) : TargetOpts(), Triple(T) { } // Out of line virtual dtor for TargetInfo. -TargetInfo::~TargetInfo() = default; +TargetInfo::~TargetInfo() {} /// getTypeName - Return the user string for the specified integer type enum. /// For example, SignedShort -> "short". diff --git a/lib/Basic/VirtualFileSystem.cpp b/lib/Basic/VirtualFileSystem.cpp index 540116f6c5..a20132b3de 100644 --- a/lib/Basic/VirtualFileSystem.cpp +++ b/lib/Basic/VirtualFileSystem.cpp @@ -84,9 +84,9 @@ bool Status::exists() const { return isStatusKnown() && Type != file_type::file_not_found; } -File::~File() = default; +File::~File() {} -FileSystem::~FileSystem() = default; +FileSystem::~FileSystem() {} ErrorOr<std::unique_ptr<MemoryBuffer>> FileSystem::getBufferForFile(const llvm::Twine &Name, int64_t FileSize, @@ -315,7 +315,7 @@ OverlayFileSystem::setCurrentWorkingDirectory(const Twine &Path) { return std::error_code(); } -clang::vfs::detail::DirIterImpl::~DirIterImpl() = default; +clang::vfs::detail::DirIterImpl::~DirIterImpl() { } namespace { class OverlayFSDirIterImpl : public clang::vfs::detail::DirIterImpl { @@ -398,7 +398,7 @@ class InMemoryNode { public: InMemoryNode(Status Stat, InMemoryNodeKind Kind) : Stat(std::move(Stat)), Kind(Kind) {} - virtual ~InMemoryNode() = default; + virtual ~InMemoryNode() {} const Status &getStatus() const { return Stat; } InMemoryNodeKind getKind() const { return Kind; } virtual std::string toString(unsigned Indent) const = 0; @@ -482,7 +482,7 @@ InMemoryFileSystem::InMemoryFileSystem(bool UseNormalizedPaths) llvm::sys::fs::perms::all_all))), UseNormalizedPaths(UseNormalizedPaths) {} -InMemoryFileSystem::~InMemoryFileSystem() = default; +InMemoryFileSystem::~InMemoryFileSystem() {} std::string InMemoryFileSystem::toString() const { return Root->toString(/*Indent=*/0); diff --git a/lib/CodeGen/CGBlocks.cpp b/lib/CodeGen/CGBlocks.cpp index b09532bbad..cfb09e72c7 100644 --- a/lib/CodeGen/CGBlocks.cpp +++ b/lib/CodeGen/CGBlocks.cpp @@ -40,7 +40,7 @@ CGBlockInfo::CGBlockInfo(const BlockDecl *block, StringRef name) } // Anchor the vtable to this translation unit. -BlockByrefHelpers::~BlockByrefHelpers() = default; +BlockByrefHelpers::~BlockByrefHelpers() {} /// Build the given block as a global block. static llvm::Constant *buildGlobalBlock(CodeGenModule &CGM, diff --git a/lib/CodeGen/CGCUDARuntime.cpp b/lib/CodeGen/CGCUDARuntime.cpp index bd8b034c81..014a5dbd46 100644 --- a/lib/CodeGen/CGCUDARuntime.cpp +++ b/lib/CodeGen/CGCUDARuntime.cpp @@ -22,7 +22,7 @@ using namespace clang; using namespace CodeGen; -CGCUDARuntime::~CGCUDARuntime() = default; +CGCUDARuntime::~CGCUDARuntime() {} RValue CGCUDARuntime::EmitCUDAKernelCallExpr(CodeGenFunction &CGF, const CUDAKernelCallExpr *E, diff --git a/lib/CodeGen/CGCXXABI.cpp b/lib/CodeGen/CGCXXABI.cpp index 5455000f21..51f1b0730b 100644 --- a/lib/CodeGen/CGCXXABI.cpp +++ b/lib/CodeGen/CGCXXABI.cpp @@ -18,7 +18,7 @@ using namespace clang; using namespace CodeGen; -CGCXXABI::~CGCXXABI() = default; +CGCXXABI::~CGCXXABI() { } void CGCXXABI::ErrorUnsupportedABI(CodeGenFunction &CGF, StringRef S) { DiagnosticsEngine &Diags = CGF.CGM.getDiags(); diff --git a/lib/CodeGen/CGCall.cpp b/lib/CodeGen/CGCall.cpp index 2b8e44ba72..bb7fdca667 100644 --- a/lib/CodeGen/CGCall.cpp +++ b/lib/CodeGen/CGCall.cpp @@ -584,7 +584,7 @@ struct TypeExpansion { const TypeExpansionKind Kind; TypeExpansion(TypeExpansionKind K) : Kind(K) {} - virtual ~TypeExpansion() = default; + virtual ~TypeExpansion() {} }; struct ConstantArrayExpansion : TypeExpansion { diff --git a/lib/CodeGen/CGObjC.cpp b/lib/CodeGen/CGObjC.cpp index 12ea96755c..c4b5b43b71 100644 --- a/lib/CodeGen/CGObjC.cpp +++ b/lib/CodeGen/CGObjC.cpp @@ -3058,4 +3058,4 @@ CodeGenFunction::EmitBlockCopyAndAutorelease(llvm::Value *Block, QualType Ty) { } -CGObjCRuntime::~CGObjCRuntime() = default; +CGObjCRuntime::~CGObjCRuntime() {} diff --git a/lib/CodeGen/CGOpenCLRuntime.cpp b/lib/CodeGen/CGOpenCLRuntime.cpp index 808406a97c..8af39ceecd 100644 --- a/lib/CodeGen/CGOpenCLRuntime.cpp +++ b/lib/CodeGen/CGOpenCLRuntime.cpp @@ -22,7 +22,7 @@ using namespace clang; using namespace CodeGen; -CGOpenCLRuntime::~CGOpenCLRuntime() = default; +CGOpenCLRuntime::~CGOpenCLRuntime() {} void CGOpenCLRuntime::EmitWorkGroupLocalVarDecl(CodeGenFunction &CGF, const VarDecl &D) { diff --git a/lib/CodeGen/CGOpenMPRuntime.h b/lib/CodeGen/CGOpenMPRuntime.h index 947f2cf7a9..dc4238cda6 100644 --- a/lib/CodeGen/CGOpenMPRuntime.h +++ b/lib/CodeGen/CGOpenMPRuntime.h @@ -369,7 +369,7 @@ private: public: explicit CGOpenMPRuntime(CodeGenModule &CGM); - virtual ~CGOpenMPRuntime() = default; + virtual ~CGOpenMPRuntime() {} virtual void clear(); /// \brief Emits outlined function for the specified OpenMP parallel directive diff --git a/lib/CodeGen/CodeGenFunction.cpp b/lib/CodeGen/CodeGenFunction.cpp index 9fbfcf424a..8801c3b3f2 100644 --- a/lib/CodeGen/CodeGenFunction.cpp +++ b/lib/CodeGen/CodeGenFunction.cpp @@ -1801,7 +1801,7 @@ Address CodeGenFunction::EmitFieldAnnotations(const FieldDecl *D, return Address(V, Addr.getAlignment()); } -CodeGenFunction::CGCapturedStmtInfo::~CGCapturedStmtInfo() = default; +CodeGenFunction::CGCapturedStmtInfo::~CGCapturedStmtInfo() { } CodeGenFunction::SanitizerScope::SanitizerScope(CodeGenFunction *CGF) : CGF(CGF) { diff --git a/lib/CodeGen/CodeGenTBAA.cpp b/lib/CodeGen/CodeGenTBAA.cpp index ee232ce773..c3c925cde2 100644 --- a/lib/CodeGen/CodeGenTBAA.cpp +++ b/lib/CodeGen/CodeGenTBAA.cpp @@ -36,7 +36,8 @@ CodeGenTBAA::CodeGenTBAA(ASTContext &Ctx, llvm::LLVMContext& VMContext, MDHelper(VMContext), Root(nullptr), Char(nullptr) { } -CodeGenTBAA::~CodeGenTBAA() = default; +CodeGenTBAA::~CodeGenTBAA() { +} llvm::MDNode *CodeGenTBAA::getRoot() { // Define the root of the tree. This identifies the tree, so that diff --git a/lib/CodeGen/TargetInfo.cpp b/lib/CodeGen/TargetInfo.cpp index aa54b0a4d4..f76fad16cc 100644 --- a/lib/CodeGen/TargetInfo.cpp +++ b/lib/CodeGen/TargetInfo.cpp @@ -66,7 +66,7 @@ Address ABIInfo::EmitMSVAArg(CodeGenFunction &CGF, Address VAListAddr, return Address::invalid(); } -ABIInfo::~ABIInfo() = default; +ABIInfo::~ABIInfo() {} static CGCXXABI::RecordArgABI getRecordArgABI(const RecordType *RT, CGCXXABI &CXXABI) { diff --git a/lib/Driver/Tool.cpp b/lib/Driver/Tool.cpp index 6ae3506e65..7142e822f1 100644 --- a/lib/Driver/Tool.cpp +++ b/lib/Driver/Tool.cpp @@ -19,4 +19,5 @@ Tool::Tool(const char *_Name, const char *_ShortName, const ToolChain &TC, ResponseSupport(_ResponseSupport), ResponseEncoding(_ResponseEncoding), ResponseFlag(_ResponseFlag) {} -Tool::~Tool() = default; +Tool::~Tool() { +} diff --git a/lib/Driver/ToolChain.cpp b/lib/Driver/ToolChain.cpp index e4611d21b6..65d4f4f4e0 100644 --- a/lib/Driver/ToolChain.cpp +++ b/lib/Driver/ToolChain.cpp @@ -72,7 +72,8 @@ ToolChain::ToolChain(const Driver &D, const llvm::Triple &T, << A->getValue() << A->getAsString(Args); } -ToolChain::~ToolChain() = default; +ToolChain::~ToolChain() { +} vfs::FileSystem &ToolChain::getVFS() const { return getDriver().getVFS(); } diff --git a/lib/Driver/ToolChains.cpp b/lib/Driver/ToolChains.cpp index 0687699e4a..6438ea7606 100644 --- a/lib/Driver/ToolChains.cpp +++ b/lib/Driver/ToolChains.cpp @@ -156,9 +156,9 @@ StringRef MachO::getMachOArchName(const ArgList &Args) const { } } -Darwin::~Darwin() = default; +Darwin::~Darwin() {} -MachO::~MachO() = default; +MachO::~MachO() {} std::string MachO::ComputeEffectiveClangTriple(const ArgList &Args, types::ID InputType) const { @@ -2102,7 +2102,7 @@ Generic_GCC::Generic_GCC(const Driver &D, const llvm::Triple &Triple, getProgramPaths().push_back(getDriver().Dir); } -Generic_GCC::~Generic_GCC() = default; +Generic_GCC::~Generic_GCC() {} Tool *Generic_GCC::getTool(Action::ActionClass AC) const { switch (AC) { @@ -2339,7 +2339,7 @@ HexagonToolChain::HexagonToolChain(const Driver &D, const llvm::Triple &Triple, GetTargetCPU(Args), InstalledDir, LibPaths); } -HexagonToolChain::~HexagonToolChain() = default; +HexagonToolChain::~HexagonToolChain() {} Tool *HexagonToolChain::buildAssembler() const { return new tools::hexagon::Assembler(*this); @@ -2642,7 +2642,7 @@ TCEToolChain::TCEToolChain(const Driver &D, const llvm::Triple &Triple, getProgramPaths().push_back(Path); } -TCEToolChain::~TCEToolChain() = default; +TCEToolChain::~TCEToolChain() {} bool TCEToolChain::IsMathErrnoDefault() const { return true; } @@ -3987,7 +3987,7 @@ MyriadToolChain::MyriadToolChain(const Driver &D, const llvm::Triple &Triple, } } -MyriadToolChain::~MyriadToolChain() = default; +MyriadToolChain::~MyriadToolChain() {} void MyriadToolChain::AddClangSystemIncludeArgs(const ArgList &DriverArgs, ArgStringList &CC1Args) const { diff --git a/lib/Format/BreakableToken.h b/lib/Format/BreakableToken.h index 506d979f2d..eb1f9fda30 100644 --- a/lib/Format/BreakableToken.h +++ b/lib/Format/BreakableToken.h @@ -36,7 +36,7 @@ public: /// \brief Contains starting character index and length of split. typedef std::pair<StringRef::size_type, unsigned> Split; - virtual ~BreakableToken() = default; + virtual ~BreakableToken() {} /// \brief Returns the number of lines in this token in the original code. virtual unsigned getLineCount() const = 0; diff --git a/lib/Format/FormatToken.cpp b/lib/Format/FormatToken.cpp index fec48605b0..b293cf25c3 100644 --- a/lib/Format/FormatToken.cpp +++ b/lib/Format/FormatToken.cpp @@ -67,7 +67,7 @@ bool FormatToken::isSimpleTypeSpecifier() const { } } -TokenRole::~TokenRole() = default; +TokenRole::~TokenRole() {} void TokenRole::precomputeFormattingInfos(const FormatToken *Token) {} diff --git a/lib/Format/UnwrappedLineFormatter.cpp b/lib/Format/UnwrappedLineFormatter.cpp index 100aedcd99..6d9b1da49b 100644 --- a/lib/Format/UnwrappedLineFormatter.cpp +++ b/lib/Format/UnwrappedLineFormatter.cpp @@ -477,7 +477,7 @@ public: UnwrappedLineFormatter *BlockFormatter) : Indenter(Indenter), Whitespaces(Whitespaces), Style(Style), BlockFormatter(BlockFormatter) {} - virtual ~LineFormatter() = default; + virtual ~LineFormatter() {} /// \brief Formats an \c AnnotatedLine and returns the penalty. /// diff --git a/lib/Format/UnwrappedLineParser.cpp b/lib/Format/UnwrappedLineParser.cpp index 2956f841e7..634dbe9708 100644 --- a/lib/Format/UnwrappedLineParser.cpp +++ b/lib/Format/UnwrappedLineParser.cpp @@ -25,7 +25,7 @@ namespace format { class FormatTokenSource { public: - virtual ~FormatTokenSource() = default; + virtual ~FormatTokenSource() {} virtual FormatToken *getNextToken() = 0; virtual unsigned getPosition() = 0; diff --git a/lib/Format/UnwrappedLineParser.h b/lib/Format/UnwrappedLineParser.h index 88c0dc5cb7..c2fa029576 100644 --- a/lib/Format/UnwrappedLineParser.h +++ b/lib/Format/UnwrappedLineParser.h @@ -51,7 +51,7 @@ struct UnwrappedLine { class UnwrappedLineConsumer { public: - virtual ~UnwrappedLineConsumer() = default; + virtual ~UnwrappedLineConsumer() {} virtual void consumeUnwrappedLine(const UnwrappedLine &Line) = 0; virtual void finishRun() = 0; }; diff --git a/lib/Frontend/ASTUnit.cpp b/lib/Frontend/ASTUnit.cpp index 1696e36418..d3d7185ea3 100644 --- a/lib/Frontend/ASTUnit.cpp +++ b/lib/Frontend/ASTUnit.cpp @@ -2841,7 +2841,7 @@ void ASTUnit::ConcurrencyState::finish() { #else // NDEBUG ASTUnit::ConcurrencyState::ConcurrencyState() { Mutex = nullptr; } -ASTUnit::ConcurrencyState::~ConcurrencyState() = default; +ASTUnit::ConcurrencyState::~ConcurrencyState() {} void ASTUnit::ConcurrencyState::start() {} void ASTUnit::ConcurrencyState::finish() {} diff --git a/lib/Frontend/CacheTokens.cpp b/lib/Frontend/CacheTokens.cpp index 9f64185555..7d2a09cd7c 100644 --- a/lib/Frontend/CacheTokens.cpp +++ b/lib/Frontend/CacheTokens.cpp @@ -47,7 +47,7 @@ class PTHEntry { Offset TokenData, PPCondData; public: - PTHEntry() = default; + PTHEntry() {} PTHEntry(Offset td, Offset ppcd) : TokenData(td), PPCondData(ppcd) {} @@ -547,7 +547,7 @@ class StatListener : public FileSystemStatCache { PTHMap &PM; public: StatListener(PTHMap &pm) : PM(pm) {} - ~StatListener() override = default; + ~StatListener() override {} LookupResult getStat(const char *Path, FileData &Data, bool isFile, std::unique_ptr<vfs::File> *F, diff --git a/lib/Frontend/CompilerInvocation.cpp b/lib/Frontend/CompilerInvocation.cpp index b5ac59990f..6e234112d7 100644 --- a/lib/Frontend/CompilerInvocation.cpp +++ b/lib/Frontend/CompilerInvocation.cpp @@ -59,7 +59,7 @@ CompilerInvocationBase::CompilerInvocationBase(const CompilerInvocationBase &X) HeaderSearchOpts(new HeaderSearchOptions(X.getHeaderSearchOpts())), PreprocessorOpts(new PreprocessorOptions(X.getPreprocessorOpts())) {} -CompilerInvocationBase::~CompilerInvocationBase() = default; +CompilerInvocationBase::~CompilerInvocationBase() {} //===----------------------------------------------------------------------===// // Deserialization (from args) diff --git a/lib/Frontend/DependencyFile.cpp b/lib/Frontend/DependencyFile.cpp index 15ddb702cb..93d4a80346 100644 --- a/lib/Frontend/DependencyFile.cpp +++ b/lib/Frontend/DependencyFile.cpp @@ -137,7 +137,7 @@ bool DependencyCollector::sawDependency(StringRef Filename, bool FromModule, (needSystemDependencies() || !IsSystem); } -DependencyCollector::~DependencyCollector() = default; +DependencyCollector::~DependencyCollector() { } void DependencyCollector::attachToPreprocessor(Preprocessor &PP) { PP.addPPCallbacks( llvm::make_unique<DepCollectorPPCallbacks>(*this, PP.getSourceManager())); diff --git a/lib/Frontend/DiagnosticRenderer.cpp b/lib/Frontend/DiagnosticRenderer.cpp index b8d4124f17..6ef9a89d86 100644 --- a/lib/Frontend/DiagnosticRenderer.cpp +++ b/lib/Frontend/DiagnosticRenderer.cpp @@ -69,7 +69,7 @@ DiagnosticRenderer::DiagnosticRenderer(const LangOptions &LangOpts, DiagnosticOptions *DiagOpts) : LangOpts(LangOpts), DiagOpts(DiagOpts), LastLevel() {} -DiagnosticRenderer::~DiagnosticRenderer() = default; +DiagnosticRenderer::~DiagnosticRenderer() {} namespace { @@ -577,7 +577,7 @@ void DiagnosticRenderer::emitMacroExpansions(SourceLocation Loc, emitSingleMacroExpansion(*I, Level, Ranges, SM); } -DiagnosticNoteRenderer::~DiagnosticNoteRenderer() = default; +DiagnosticNoteRenderer::~DiagnosticNoteRenderer() {} void DiagnosticNoteRenderer::emitIncludeLocation(SourceLocation Loc, PresumedLoc PLoc, diff --git a/lib/Frontend/FrontendAction.cpp b/lib/Frontend/FrontendAction.cpp index 3c6c7d5a44..ecef92e0a7 100644 --- a/lib/Frontend/FrontendAction.cpp +++ b/lib/Frontend/FrontendAction.cpp @@ -126,7 +126,7 @@ public: FrontendAction::FrontendAction() : Instance(nullptr) {} -FrontendAction::~FrontendAction() = default; +FrontendAction::~FrontendAction() {} void FrontendAction::setCurrentInput(const FrontendInputFile &CurrentInput, std::unique_ptr<ASTUnit> AST) { diff --git a/lib/Frontend/MultiplexConsumer.cpp b/lib/Frontend/MultiplexConsumer.cpp index 226e33da45..91ee100f63 100644 --- a/lib/Frontend/MultiplexConsumer.cpp +++ b/lib/Frontend/MultiplexConsumer.cpp @@ -262,7 +262,7 @@ MultiplexConsumer::MultiplexConsumer( } } -MultiplexConsumer::~MultiplexConsumer() = default; +MultiplexConsumer::~MultiplexConsumer() {} void MultiplexConsumer::Initialize(ASTContext &Context) { for (auto &Consumer : Consumers) diff --git a/lib/Frontend/Rewrite/FixItRewriter.cpp b/lib/Frontend/Rewrite/FixItRewriter.cpp index 8daf2ce41b..dc787ac955 100644 --- a/lib/Frontend/Rewrite/FixItRewriter.cpp +++ b/lib/Frontend/Rewrite/FixItRewriter.cpp @@ -200,4 +200,4 @@ void FixItRewriter::Diag(SourceLocation Loc, unsigned DiagID) { Diags.setClient(this, false); } -FixItOptions::~FixItOptions() = default; +FixItOptions::~FixItOptions() {} diff --git a/lib/Frontend/Rewrite/FrontendActions.cpp b/lib/Frontend/Rewrite/FrontendActions.cpp index d3a22ce9ad..8cf8adf37e 100644 --- a/lib/Frontend/Rewrite/FrontendActions.cpp +++ b/lib/Frontend/Rewrite/FrontendActions.cpp @@ -38,7 +38,7 @@ HTMLPrintAction::CreateASTConsumer(CompilerInstance &CI, StringRef InFile) { } FixItAction::FixItAction() {} -FixItAction::~FixItAction() = default; +FixItAction::~FixItAction() {} std::unique_ptr<ASTConsumer> FixItAction::CreateASTConsumer(CompilerInstance &CI, StringRef InFile) { diff --git a/lib/Frontend/Rewrite/RewriteModernObjC.cpp b/lib/Frontend/Rewrite/RewriteModernObjC.cpp index 3523783b5c..2902ba78c4 100644 --- a/lib/Frontend/Rewrite/RewriteModernObjC.cpp +++ b/lib/Frontend/Rewrite/RewriteModernObjC.cpp @@ -244,7 +244,7 @@ namespace { DiagnosticsEngine &D, const LangOptions &LOpts, bool silenceMacroWarn, bool LineInfo); - ~RewriteModernObjC() override = default; + ~RewriteModernObjC() override {} void HandleTranslationUnit(ASTContext &C) override; diff --git a/lib/Frontend/Rewrite/RewriteObjC.cpp b/lib/Frontend/Rewrite/RewriteObjC.cpp index 9d9be2cbc3..204820b304 100644 --- a/lib/Frontend/Rewrite/RewriteObjC.cpp +++ b/lib/Frontend/Rewrite/RewriteObjC.cpp @@ -193,7 +193,7 @@ namespace { DiagnosticsEngine &D, const LangOptions &LOpts, bool silenceMacroWarn); - ~RewriteObjC() override = default; + ~RewriteObjC() override {} void HandleTranslationUnit(ASTContext &C) override; @@ -512,7 +512,7 @@ namespace { D, LOpts, silenceMacroWarn) {} - ~RewriteObjCFragileABI() override = default; + ~RewriteObjCFragileABI() override {} void Initialize(ASTContext &context) override; // Rewriting metadata diff --git a/lib/Frontend/SerializedDiagnosticPrinter.cpp b/lib/Frontend/SerializedDiagnosticPrinter.cpp index 1cf886ce7b..1bf10d2769 100644 --- a/lib/Frontend/SerializedDiagnosticPrinter.cpp +++ b/lib/Frontend/SerializedDiagnosticPrinter.cpp @@ -62,7 +62,7 @@ public: DiagnosticOptions *DiagOpts) : DiagnosticNoteRenderer(LangOpts, DiagOpts), Writer(Writer) {} - ~SDiagsRenderer() override = default; + ~SDiagsRenderer() override {} protected: void emitDiagnosticMessage(SourceLocation Loc, @@ -159,7 +159,7 @@ public: EmitPreamble(); } - ~SDiagsWriter() override = default; + ~SDiagsWriter() override {} void HandleDiagnostic(DiagnosticsEngine::Level DiagLevel, const Diagnostic &Info) override; diff --git a/lib/Frontend/TextDiagnostic.cpp b/lib/Frontend/TextDiagnostic.cpp index 938fe58ea8..d4e156d445 100644 --- a/lib/Frontend/TextDiagnostic.cpp +++ b/lib/Frontend/TextDiagnostic.cpp @@ -669,7 +669,7 @@ TextDiagnostic::TextDiagnostic(raw_ostream &OS, DiagnosticOptions *DiagOpts) : DiagnosticRenderer(LangOpts, DiagOpts), OS(OS) {} -TextDiagnostic::~TextDiagnostic() = default; +TextDiagnostic::~TextDiagnostic() {} void TextDiagnostic::emitDiagnosticMessage(SourceLocation Loc, diff --git a/lib/Index/CommentToXML.cpp b/lib/Index/CommentToXML.cpp index df103db1de..15f1696cbe 100644 --- a/lib/Index/CommentToXML.cpp +++ b/lib/Index/CommentToXML.cpp @@ -1128,7 +1128,7 @@ void CommentASTToXMLConverter::appendToResultWithCDATAEscaping(StringRef S) { } CommentToXMLConverter::CommentToXMLConverter() : FormatInMemoryUniqueId(0) {} -CommentToXMLConverter::~CommentToXMLConverter() = default; +CommentToXMLConverter::~CommentToXMLConverter() {} void CommentToXMLConverter::convertCommentToHTML(const FullComment *FC, SmallVectorImpl<char> &HTML, diff --git a/lib/Lex/HeaderSearch.cpp b/lib/Lex/HeaderSearch.cpp index 79564ca2d5..469d782deb 100644 --- a/lib/Lex/HeaderSearch.cpp +++ b/lib/Lex/HeaderSearch.cpp @@ -50,7 +50,7 @@ HeaderFileInfo::getControllingMacro(ExternalPreprocessorSource *External) { return ControllingMacro; } -ExternalHeaderFileInfoSource::~ExternalHeaderFileInfoSource() = default; +ExternalHeaderFileInfoSource::~ExternalHeaderFileInfoSource() {} HeaderSearch::HeaderSearch(IntrusiveRefCntPtr<HeaderSearchOptions> HSOpts, SourceManager &SourceMgr, DiagnosticsEngine &Diags, diff --git a/lib/Lex/PPLexerChange.cpp b/lib/Lex/PPLexerChange.cpp index 8f5171880a..c231e18eec 100644 --- a/lib/Lex/PPLexerChange.cpp +++ b/lib/Lex/PPLexerChange.cpp @@ -24,7 +24,7 @@ #include "llvm/Support/Path.h" using namespace clang; -PPCallbacks::~PPCallbacks() = default; +PPCallbacks::~PPCallbacks() {} //===----------------------------------------------------------------------===// // Miscellaneous Methods. diff --git a/lib/Lex/PTHLexer.cpp b/lib/Lex/PTHLexer.cpp index f1a625a882..5f63d35c5b 100644 --- a/lib/Lex/PTHLexer.cpp +++ b/lib/Lex/PTHLexer.cpp @@ -424,7 +424,8 @@ PTHManager::PTHManager( StringIdLookup(std::move(stringIdLookup)), NumIds(numIds), PP(nullptr), SpellingBase(spellingBase), OriginalSourceFile(originalSourceFile) {} -PTHManager::~PTHManager() = default; +PTHManager::~PTHManager() { +} static void InvalidPTH(DiagnosticsEngine &Diags, const char *Msg) { Diags.Report(Diags.getCustomDiagID(DiagnosticsEngine::Error, "%0")) << Msg; diff --git a/lib/Lex/Pragma.cpp b/lib/Lex/Pragma.cpp index 1f158cf5e3..3134790ccb 100644 --- a/lib/Lex/Pragma.cpp +++ b/lib/Lex/Pragma.cpp @@ -31,7 +31,8 @@ using namespace clang; #include "llvm/Support/raw_ostream.h" // Out-of-line destructor to provide a home for the class. -PragmaHandler::~PragmaHandler() = default; +PragmaHandler::~PragmaHandler() { +} //===----------------------------------------------------------------------===// // EmptyPragmaHandler Implementation. diff --git a/lib/Lex/PreprocessingRecord.cpp b/lib/Lex/PreprocessingRecord.cpp index d01dfca8b6..32e6de69f0 100644 --- a/lib/Lex/PreprocessingRecord.cpp +++ b/lib/Lex/PreprocessingRecord.cpp @@ -19,7 +19,7 @@ using namespace clang; -ExternalPreprocessingRecordSource::~ExternalPreprocessingRecordSource() = default; +ExternalPreprocessingRecordSource::~ExternalPreprocessingRecordSource() { } InclusionDirective::InclusionDirective(PreprocessingRecord &PPRec, diff --git a/lib/Lex/Preprocessor.cpp b/lib/Lex/Preprocessor.cpp index 7b1f78a510..869890b46c 100644 --- a/lib/Lex/Preprocessor.cpp +++ b/lib/Lex/Preprocessor.cpp @@ -53,7 +53,7 @@ using namespace clang; //===----------------------------------------------------------------------===// -ExternalPreprocessorSource::~ExternalPreprocessorSource() = default; +ExternalPreprocessorSource::~ExternalPreprocessorSource() { } Preprocessor::Preprocessor(IntrusiveRefCntPtr<PreprocessorOptions> PPOpts, DiagnosticsEngine &diags, LangOptions &opts, @@ -897,11 +897,11 @@ bool Preprocessor::HandleComment(Token &result, SourceRange Comment) { return true; } -ModuleLoader::~ModuleLoader() = default; +ModuleLoader::~ModuleLoader() { } -CommentHandler::~CommentHandler() = default; +CommentHandler::~CommentHandler() { } -CodeCompletionHandler::~CodeCompletionHandler() = default; +CodeCompletionHandler::~CodeCompletionHandler() { } void Preprocessor::createPreprocessingRecord() { if (Record) diff --git a/lib/Parse/ParseCXXInlineMethods.cpp b/lib/Parse/ParseCXXInlineMethods.cpp index 82fef5ace4..ab1f97d31a 100644 --- a/lib/Parse/ParseCXXInlineMethods.cpp +++ b/lib/Parse/ParseCXXInlineMethods.cpp @@ -222,7 +222,7 @@ void Parser::ParseCXXNonStaticMemberInitializer(Decl *VarD) { Toks.push_back(Eof); } -Parser::LateParsedDeclaration::~LateParsedDeclaration() = default; +Parser::LateParsedDeclaration::~LateParsedDeclaration() {} void Parser::LateParsedDeclaration::ParseLexedMethodDeclarations() {} void Parser::LateParsedDeclaration::ParseLexedMemberInitializers() {} void Parser::LateParsedDeclaration::ParseLexedMethodDefs() {} diff --git a/lib/Rewrite/TokenRewriter.cpp b/lib/Rewrite/TokenRewriter.cpp index cc2ed1350a..494defdeda 100644 --- a/lib/Rewrite/TokenRewriter.cpp +++ b/lib/Rewrite/TokenRewriter.cpp @@ -46,7 +46,8 @@ TokenRewriter::TokenRewriter(FileID FID, SourceManager &SM, } } -TokenRewriter::~TokenRewriter() = default; +TokenRewriter::~TokenRewriter() { +} /// RemapIterator - Convert from token_iterator (a const iterator) to diff --git a/lib/Sema/AttributeList.cpp b/lib/Sema/AttributeList.cpp index 02986deacc..3c61c95ad8 100644 --- a/lib/Sema/AttributeList.cpp +++ b/lib/Sema/AttributeList.cpp @@ -44,7 +44,7 @@ AttributeFactory::AttributeFactory() { // Go ahead and configure all the inline capacity. This is just a memset. FreeLists.resize(InlineFreeListsCapacity); } -AttributeFactory::~AttributeFactory() = default; +AttributeFactory::~AttributeFactory() {} static size_t getFreeListIndexForSize(size_t size) { assert(size >= sizeof(AttributeList)); diff --git a/lib/Sema/CodeCompleteConsumer.cpp b/lib/Sema/CodeCompleteConsumer.cpp index 708d9f5875..18e9a59116 100644 --- a/lib/Sema/CodeCompleteConsumer.cpp +++ b/lib/Sema/CodeCompleteConsumer.cpp @@ -426,7 +426,7 @@ CodeCompleteConsumer::OverloadCandidate::getFunctionType() const { // Code completion consumer implementation //===----------------------------------------------------------------------===// -CodeCompleteConsumer::~CodeCompleteConsumer() = default; +CodeCompleteConsumer::~CodeCompleteConsumer() { } void PrintingCodeCompleteConsumer::ProcessCodeCompleteResults(Sema &SemaRef, diff --git a/lib/Sema/MultiplexExternalSemaSource.cpp b/lib/Sema/MultiplexExternalSemaSource.cpp index 5d0effc5a5..0f93421ac2 100644 --- a/lib/Sema/MultiplexExternalSemaSource.cpp +++ b/lib/Sema/MultiplexExternalSemaSource.cpp @@ -28,7 +28,7 @@ MultiplexExternalSemaSource::MultiplexExternalSemaSource(ExternalSemaSource &s1, } // pin the vtable here. -MultiplexExternalSemaSource::~MultiplexExternalSemaSource() = default; +MultiplexExternalSemaSource::~MultiplexExternalSemaSource() {} ///\brief Appends new source to the source list. /// diff --git a/lib/Sema/ScopeInfo.cpp b/lib/Sema/ScopeInfo.cpp index a06375816f..ba8c3642bc 100644 --- a/lib/Sema/ScopeInfo.cpp +++ b/lib/Sema/ScopeInfo.cpp @@ -232,6 +232,6 @@ void LambdaScopeInfo::getPotentialVariableCapture(unsigned Idx, VarDecl *&VD, assert(VD); } -FunctionScopeInfo::~FunctionScopeInfo() = default; -BlockScopeInfo::~BlockScopeInfo() = default; -CapturedRegionScopeInfo::~CapturedRegionScopeInfo() = default; +FunctionScopeInfo::~FunctionScopeInfo() { } +BlockScopeInfo::~BlockScopeInfo() { } +CapturedRegionScopeInfo::~CapturedRegionScopeInfo() { } diff --git a/lib/Sema/Sema.cpp b/lib/Sema/Sema.cpp index bac323712e..300e2f040a 100644 --- a/lib/Sema/Sema.cpp +++ b/lib/Sema/Sema.cpp @@ -1255,7 +1255,7 @@ void Sema::ActOnComment(SourceRange Comment) { } // Pin this vtable to this file. -ExternalSemaSource::~ExternalSemaSource() = default; +ExternalSemaSource::~ExternalSemaSource() {} void ExternalSemaSource::ReadMethodPool(Selector Sel) { } diff --git a/lib/Sema/SemaDeclCXX.cpp b/lib/Sema/SemaDeclCXX.cpp index 1d027d5f07..588ae3d88e 100644 --- a/lib/Sema/SemaDeclCXX.cpp +++ b/lib/Sema/SemaDeclCXX.cpp @@ -9567,8 +9567,8 @@ protected: } public: - ExprBuilder() = default; - virtual ~ExprBuilder() = default; + ExprBuilder() {} + virtual ~ExprBuilder() {} virtual Expr *build(Sema &S, SourceLocation Loc) const = 0; }; diff --git a/lib/Sema/SemaLookup.cpp b/lib/Sema/SemaLookup.cpp index 556e5bb8dc..6fbe0da8c9 100644 --- a/lib/Sema/SemaLookup.cpp +++ b/lib/Sema/SemaLookup.cpp @@ -3172,7 +3172,7 @@ void Sema::ArgumentDependentLookup(DeclarationName Name, SourceLocation Loc, //---------------------------------------------------------------------------- // Search for all visible declarations. //---------------------------------------------------------------------------- -VisibleDeclConsumer::~VisibleDeclConsumer() = default; +VisibleDeclConsumer::~VisibleDeclConsumer() { } bool VisibleDeclConsumer::includeHiddenDecls() const { return false; } diff --git a/lib/Sema/SemaOpenMP.cpp b/lib/Sema/SemaOpenMP.cpp index db658de7d9..c4e1b26e7c 100644 --- a/lib/Sema/SemaOpenMP.cpp +++ b/lib/Sema/SemaOpenMP.cpp @@ -56,7 +56,7 @@ private: ArrayRef<T> Arr; }; struct MatchesAlways { - MatchesAlways() = default; + MatchesAlways() {} template <class T> bool operator()(T) { return true; } }; diff --git a/lib/Sema/SemaPseudoObject.cpp b/lib/Sema/SemaPseudoObject.cpp index f8974adfac..8998a7e8bc 100644 --- a/lib/Sema/SemaPseudoObject.cpp +++ b/lib/Sema/SemaPseudoObject.cpp @@ -202,7 +202,7 @@ namespace { : S(S), ResultIndex(PseudoObjectExpr::NoResult), GenericLoc(genericLoc) {} - virtual ~PseudoOpBuilder() = default; + virtual ~PseudoOpBuilder() {} /// Add a normal semantic expression. void addSemanticExpr(Expr *semantic) { diff --git a/lib/Sema/TreeTransform.h b/lib/Sema/TreeTransform.h index 9378c98899..31d69cbac3 100644 --- a/lib/Sema/TreeTransform.h +++ b/lib/Sema/TreeTransform.h @@ -3600,7 +3600,7 @@ public: const TemplateArgumentLoc *operator->() const { return &Arg; } }; - TemplateArgumentLocInventIterator() = default; + TemplateArgumentLocInventIterator() { } explicit TemplateArgumentLocInventIterator(TreeTransform<Derived> &Self, InputIterator Iter) @@ -5249,7 +5249,7 @@ QualType TreeTransform<Derived>::TransformAtomicType(TypeLocBuilder &TLB, }; - TemplateArgumentLocContainerIterator() = default; + TemplateArgumentLocContainerIterator() {} TemplateArgumentLocContainerIterator(ArgLocContainer &Container, unsigned Index) diff --git a/lib/Serialization/ASTCommon.cpp b/lib/Serialization/ASTCommon.cpp index 3839fdd187..ec1a936684 100644 --- a/lib/Serialization/ASTCommon.cpp +++ b/lib/Serialization/ASTCommon.cpp @@ -21,7 +21,7 @@ using namespace clang; // Give ASTDeserializationListener's VTable a home. -ASTDeserializationListener::~ASTDeserializationListener() = default; +ASTDeserializationListener::~ASTDeserializationListener() { } serialization::TypeIdx serialization::TypeIdxFromBuiltin(const BuiltinType *BT) { diff --git a/lib/Serialization/ASTReader.cpp b/lib/Serialization/ASTReader.cpp index 5e3d7622fa..59347de9c1 100644 --- a/lib/Serialization/ASTReader.cpp +++ b/lib/Serialization/ASTReader.cpp @@ -164,7 +164,7 @@ bool ChainedASTReaderListener::visitInputFile(StringRef Filename, // PCH validator implementation //===----------------------------------------------------------------------===// -ASTReaderListener::~ASTReaderListener() = default; +ASTReaderListener::~ASTReaderListener() {} /// \brief Compare the given set of language options against an existing set of /// language options. diff --git a/lib/Serialization/GeneratePCH.cpp b/lib/Serialization/GeneratePCH.cpp index 0b9acbe5ff..9de2fdb75a 100644 --- a/lib/Serialization/GeneratePCH.cpp +++ b/lib/Serialization/GeneratePCH.cpp @@ -34,7 +34,8 @@ PCHGenerator::PCHGenerator(const Preprocessor &PP, StringRef OutputFile, Buffer->IsComplete = false; } -PCHGenerator::~PCHGenerator() = default; +PCHGenerator::~PCHGenerator() { +} void PCHGenerator::HandleTranslationUnit(ASTContext &Ctx) { // Don't create a PCH if there were fatal failures during module loading. diff --git a/lib/Serialization/MultiOnDiskHashTable.h b/lib/Serialization/MultiOnDiskHashTable.h index 6b81f1eb3d..04dea83169 100644 --- a/lib/Serialization/MultiOnDiskHashTable.h +++ b/lib/Serialization/MultiOnDiskHashTable.h @@ -161,7 +161,7 @@ private: friend class MultiOnDiskHashTableGenerator; public: - MultiOnDiskHashTable() = default; + MultiOnDiskHashTable() {} MultiOnDiskHashTable(MultiOnDiskHashTable &&O) : Tables(std::move(O.Tables)), PendingOverrides(std::move(O.PendingOverrides)) { diff --git a/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp b/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp index 8e304f7162..5d8baf6ba7 100644 --- a/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp +++ b/lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp @@ -142,7 +142,7 @@ public: : cfg(cfg), Ctx(ctx), BR(br), Checker(checker), AC(ac), Parents(parents), Escaped(escaped), currentBlock(nullptr) {} - ~DeadStoreObs() override = default; + ~DeadStoreObs() override {} bool isLive(const LiveVariables::LivenessValues &Live, const VarDecl *D) { if (Live.isLive(D)) diff --git a/lib/StaticAnalyzer/Core/BugReporter.cpp b/lib/StaticAnalyzer/Core/BugReporter.cpp index 1b17e7a6e2..56d57c5014 100644 --- a/lib/StaticAnalyzer/Core/BugReporter.cpp +++ b/lib/StaticAnalyzer/Core/BugReporter.cpp @@ -46,7 +46,7 @@ STATISTIC(MaxValidBugClassSize, "The maximum number of bug reports in the same equivalence class " "where at least one report is valid (not suppressed)"); -BugReporterVisitor::~BugReporterVisitor() = default; +BugReporterVisitor::~BugReporterVisitor() {} void BugReporterContext::anchor() {} @@ -2731,9 +2731,9 @@ PathDiagnosticLocation BugReport::getLocation(const SourceManager &SM) const { // Methods for BugReporter and subclasses. //===----------------------------------------------------------------------===// -BugReportEquivClass::~BugReportEquivClass() = default; -GRBugReporter::~GRBugReporter() = default; -BugReporterData::~BugReporterData() = default; +BugReportEquivClass::~BugReportEquivClass() { } +GRBugReporter::~GRBugReporter() { } +BugReporterData::~BugReporterData() {} ExplodedGraph &GRBugReporter::getGraph() { return Eng.getGraph(); } diff --git a/lib/StaticAnalyzer/Core/ConstraintManager.cpp b/lib/StaticAnalyzer/Core/ConstraintManager.cpp index cf35f9879f..b7db8333aa 100644 --- a/lib/StaticAnalyzer/Core/ConstraintManager.cpp +++ b/lib/StaticAnalyzer/Core/ConstraintManager.cpp @@ -16,7 +16,7 @@ using namespace clang; using namespace ento; -ConstraintManager::~ConstraintManager() = default; +ConstraintManager::~ConstraintManager() {} static DefinedSVal getLocFromSymbol(const ProgramStateRef &State, SymbolRef Sym) { diff --git a/lib/StaticAnalyzer/Core/CoreEngine.cpp b/lib/StaticAnalyzer/Core/CoreEngine.cpp index 58f81ebc53..39cf7e7717 100644 --- a/lib/StaticAnalyzer/Core/CoreEngine.cpp +++ b/lib/StaticAnalyzer/Core/CoreEngine.cpp @@ -38,7 +38,7 @@ STATISTIC(NumPathsExplored, // Worklist classes for exploration of reachable states. //===----------------------------------------------------------------------===// -WorkList::Visitor::~Visitor() = default; +WorkList::Visitor::~Visitor() {} namespace { class DFS : public WorkList { @@ -100,7 +100,7 @@ public: // Place the dstor for WorkList here because it contains virtual member // functions, and we the code for the dstor generated in one compilation unit. -WorkList::~WorkList() = default; +WorkList::~WorkList() {} WorkList *WorkList::makeDFS() { return new DFS(); } WorkList *WorkList::makeBFS() { return new BFS(); } diff --git a/lib/StaticAnalyzer/Core/ExplodedGraph.cpp b/lib/StaticAnalyzer/Core/ExplodedGraph.cpp index b9dc1c363c..8a09720b2a 100644 --- a/lib/StaticAnalyzer/Core/ExplodedGraph.cpp +++ b/lib/StaticAnalyzer/Core/ExplodedGraph.cpp @@ -31,7 +31,7 @@ using namespace ento; //===----------------------------------------------------------------------===// // An out of line virtual method to provide a home for the class vtable. -ExplodedNode::Auditor::~Auditor() = default; +ExplodedNode::Auditor::~Auditor() {} #ifndef NDEBUG static ExplodedNode::Auditor* NodeAuditor = nullptr; @@ -50,7 +50,7 @@ void ExplodedNode::SetAuditor(ExplodedNode::Auditor* A) { ExplodedGraph::ExplodedGraph() : NumNodes(0), ReclaimNodeInterval(0) {} -ExplodedGraph::~ExplodedGraph() = default; +ExplodedGraph::~ExplodedGraph() {} //===----------------------------------------------------------------------===// // Node reclamation. diff --git a/lib/StaticAnalyzer/Core/MemRegion.cpp b/lib/StaticAnalyzer/Core/MemRegion.cpp index 7946db4d38..d41fed0619 100644 --- a/lib/StaticAnalyzer/Core/MemRegion.cpp +++ b/lib/StaticAnalyzer/Core/MemRegion.cpp @@ -135,11 +135,12 @@ RegionTy* MemRegionManager::getSubRegion(const A1 a1, const A2 a2, const A3 a3, // Object destruction. //===----------------------------------------------------------------------===// -MemRegion::~MemRegion() = default; +MemRegion::~MemRegion() {} -// All regions and their data are BumpPtrAllocated. No need to call their -// destructors. -MemRegionManager::~MemRegionManager() = default; +MemRegionManager::~MemRegionManager() { + // All regions and their data are BumpPtrAllocated. No need to call + // their destructors. +} //===----------------------------------------------------------------------===// // Basic methods. diff --git a/lib/StaticAnalyzer/Core/PathDiagnostic.cpp b/lib/StaticAnalyzer/Core/PathDiagnostic.cpp index e5b473212e..504df30de8 100644 --- a/lib/StaticAnalyzer/Core/PathDiagnostic.cpp +++ b/lib/StaticAnalyzer/Core/PathDiagnostic.cpp @@ -55,11 +55,11 @@ PathDiagnosticPiece::PathDiagnosticPiece(StringRef s, PathDiagnosticPiece::PathDiagnosticPiece(Kind k, DisplayHint hint) : kind(k), Hint(hint), LastInMainSourceFile(false) {} -PathDiagnosticPiece::~PathDiagnosticPiece() = default; -PathDiagnosticEventPiece::~PathDiagnosticEventPiece() = default; -PathDiagnosticCallPiece::~PathDiagnosticCallPiece() = default; -PathDiagnosticControlFlowPiece::~PathDiagnosticControlFlowPiece() = default; -PathDiagnosticMacroPiece::~PathDiagnosticMacroPiece() = default; +PathDiagnosticPiece::~PathDiagnosticPiece() {} +PathDiagnosticEventPiece::~PathDiagnosticEventPiece() {} +PathDiagnosticCallPiece::~PathDiagnosticCallPiece() {} +PathDiagnosticControlFlowPiece::~PathDiagnosticControlFlowPiece() {} +PathDiagnosticMacroPiece::~PathDiagnosticMacroPiece() {} void PathPieces::flattenTo(PathPieces &Primary, PathPieces &Current, @@ -103,7 +103,7 @@ void PathPieces::flattenTo(PathPieces &Primary, PathPieces &Current, } -PathDiagnostic::~PathDiagnostic() = default; +PathDiagnostic::~PathDiagnostic() {} PathDiagnostic::PathDiagnostic(StringRef CheckName, const Decl *declWithIssue, StringRef bugtype, StringRef verboseDesc, @@ -1119,7 +1119,7 @@ void PathDiagnostic::FullProfile(llvm::FoldingSetNodeID &ID) const { ID.AddString(*I); } -StackHintGenerator::~StackHintGenerator() = default; +StackHintGenerator::~StackHintGenerator() {} std::string StackHintGeneratorForSymbol::getMessage(const ExplodedNode *N){ ProgramPoint P = N->getLocation(); diff --git a/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp b/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp index 557774a894..59ed921c25 100644 --- a/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp +++ b/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp @@ -37,7 +37,7 @@ namespace { const LangOptions &LangOpts, bool supportsMultipleFiles); - ~PlistDiagnostics() override = default; + ~PlistDiagnostics() override {} void FlushDiagnosticsImpl(std::vector<const PathDiagnostic *> &Diags, FilesMade *filesMade) override; diff --git a/lib/StaticAnalyzer/Core/SimpleConstraintManager.cpp b/lib/StaticAnalyzer/Core/SimpleConstraintManager.cpp index 9ed1aa5057..4051242434 100644 --- a/lib/StaticAnalyzer/Core/SimpleConstraintManager.cpp +++ b/lib/StaticAnalyzer/Core/SimpleConstraintManager.cpp @@ -21,7 +21,7 @@ namespace clang { namespace ento { -SimpleConstraintManager::~SimpleConstraintManager() = default; +SimpleConstraintManager::~SimpleConstraintManager() {} bool SimpleConstraintManager::canReasonAbout(SVal X) const { Optional<nonloc::SymbolVal> SymVal = X.getAs<nonloc::SymbolVal>(); diff --git a/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp b/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp index 6e18941f07..a704ce2245 100644 --- a/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp +++ b/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp @@ -29,7 +29,7 @@ public: SimpleSValBuilder(llvm::BumpPtrAllocator &alloc, ASTContext &context, ProgramStateManager &stateMgr) : SValBuilder(alloc, context, stateMgr) {} - ~SimpleSValBuilder() override = default; + ~SimpleSValBuilder() override {} SVal evalMinus(NonLoc val) override; SVal evalComplement(NonLoc val) override; diff --git a/lib/StaticAnalyzer/Core/Store.cpp b/lib/StaticAnalyzer/Core/Store.cpp index 245f6ad26d..7cdb55a597 100644 --- a/lib/StaticAnalyzer/Core/Store.cpp +++ b/lib/StaticAnalyzer/Core/Store.cpp @@ -489,7 +489,7 @@ SVal StoreManager::getLValueElement(QualType elementType, NonLoc Offset, Ctx)); } -StoreManager::BindingsHandler::~BindingsHandler() = default; +StoreManager::BindingsHandler::~BindingsHandler() {} bool StoreManager::FindUniqueBinding::HandleBinding(StoreManager& SMgr, Store store, diff --git a/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp b/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp index 13af3c1c71..ca92ffe8f0 100644 --- a/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp +++ b/lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp @@ -92,7 +92,7 @@ class ClangDiagPathDiagConsumer : public PathDiagnosticConsumer { public: ClangDiagPathDiagConsumer(DiagnosticsEngine &Diag) : Diag(Diag), IncludePath(false) {} - ~ClangDiagPathDiagConsumer() override = default; + ~ClangDiagPathDiagConsumer() override {} StringRef getName() const override { return "ClangDiags"; } bool supportsLogicalOpControlFlow() const override { return true; } diff --git a/lib/Tooling/CompilationDatabase.cpp b/lib/Tooling/CompilationDatabase.cpp index cdbfe5190a..957e40137e 100644 --- a/lib/Tooling/CompilationDatabase.cpp +++ b/lib/Tooling/CompilationDatabase.cpp @@ -32,7 +32,7 @@ using namespace clang; using namespace tooling; -CompilationDatabase::~CompilationDatabase() = default; +CompilationDatabase::~CompilationDatabase() {} std::unique_ptr<CompilationDatabase> CompilationDatabase::loadFromDirectory(StringRef BuildDirectory, @@ -106,7 +106,7 @@ CompilationDatabase::autoDetectFromDirectory(StringRef SourceDir, return DB; } -CompilationDatabasePlugin::~CompilationDatabasePlugin() = default; +CompilationDatabasePlugin::~CompilationDatabasePlugin() {} namespace { // Helper for recursively searching through a chain of actions and collecting diff --git a/lib/Tooling/Tooling.cpp b/lib/Tooling/Tooling.cpp index 53f6e06101..220b62558b 100644 --- a/lib/Tooling/Tooling.cpp +++ b/lib/Tooling/Tooling.cpp @@ -37,9 +37,9 @@ namespace clang { namespace tooling { -ToolAction::~ToolAction() = default; +ToolAction::~ToolAction() {} -FrontendActionFactory::~FrontendActionFactory() = default; +FrontendActionFactory::~FrontendActionFactory() {} // FIXME: This file contains structural duplication with other parts of the // code that sets up a compiler to run tools on it, and we should refactor @@ -319,7 +319,7 @@ ClangTool::ClangTool(const CompilationDatabase &Compilations, appendArgumentsAdjuster(getClangSyntaxOnlyAdjuster()); } -ClangTool::~ClangTool() = default; +ClangTool::~ClangTool() {} void ClangTool::mapVirtualFile(StringRef FilePath, StringRef Content) { MappedFileContents.push_back(std::make_pair(FilePath, Content)); diff --git a/tools/diagtool/DiagTool.cpp b/tools/diagtool/DiagTool.cpp index a8f4e1e37f..0e4d8088c6 100644 --- a/tools/diagtool/DiagTool.cpp +++ b/tools/diagtool/DiagTool.cpp @@ -22,7 +22,7 @@ DiagTool::DiagTool(llvm::StringRef toolCmd, llvm::StringRef toolDesc) : cmd(toolCmd), description(toolDesc) {} -DiagTool::~DiagTool() = default; +DiagTool::~DiagTool() {} typedef llvm::StringMap<DiagTool *> ToolMap; static inline ToolMap *getTools(void *v) { return static_cast<ToolMap*>(v); } diff --git a/tools/libclang/CIndexDiagnostic.cpp b/tools/libclang/CIndexDiagnostic.cpp index 68f0353714..9ba36a6a0f 100644 --- a/tools/libclang/CIndexDiagnostic.cpp +++ b/tools/libclang/CIndexDiagnostic.cpp @@ -30,14 +30,14 @@ using namespace clang::cxloc; using namespace clang::cxdiag; using namespace llvm; -CXDiagnosticSetImpl::~CXDiagnosticSetImpl() = default; +CXDiagnosticSetImpl::~CXDiagnosticSetImpl() {} void CXDiagnosticSetImpl::appendDiagnostic(std::unique_ptr<CXDiagnosticImpl> D) { Diagnostics.push_back(std::move(D)); } -CXDiagnosticImpl::~CXDiagnosticImpl() = default; +CXDiagnosticImpl::~CXDiagnosticImpl() {} namespace { class CXDiagnosticCustomNoteImpl : public CXDiagnosticImpl { @@ -48,7 +48,7 @@ public: : CXDiagnosticImpl(CustomNoteDiagnosticKind), Message(Msg), Loc(L) {} - ~CXDiagnosticCustomNoteImpl() override = default; + ~CXDiagnosticCustomNoteImpl() override {} CXDiagnosticSeverity getSeverity() const override { return CXDiagnostic_Note; @@ -92,7 +92,7 @@ public: : DiagnosticNoteRenderer(LangOpts, DiagOpts), CurrentSet(mainSet), MainSet(mainSet) {} - ~CXDiagnosticRenderer() override = default; + ~CXDiagnosticRenderer() override {} void beginDiagnostic(DiagOrStoredDiag D, DiagnosticsEngine::Level Level) override { diff --git a/tools/libclang/CIndexDiagnostic.h b/tools/libclang/CIndexDiagnostic.h index 4f005e5aec..9f406987eb 100644 --- a/tools/libclang/CIndexDiagnostic.h +++ b/tools/libclang/CIndexDiagnostic.h @@ -117,7 +117,7 @@ struct CXStoredDiagnostic : public CXDiagnosticImpl { : CXDiagnosticImpl(StoredDiagnosticKind), Diag(Diag), LangOpts(LangOpts) { } - ~CXStoredDiagnostic() override = default; + ~CXStoredDiagnostic() override {} /// \brief Return the severity of the diagnostic. CXDiagnosticSeverity getSeverity() const override; diff --git a/tools/libclang/CXLoadedDiagnostic.cpp b/tools/libclang/CXLoadedDiagnostic.cpp index 7f45d85fe1..2c10d34844 100644 --- a/tools/libclang/CXLoadedDiagnostic.cpp +++ b/tools/libclang/CXLoadedDiagnostic.cpp @@ -38,7 +38,7 @@ namespace { class CXLoadedDiagnosticSetImpl : public CXDiagnosticSetImpl { public: CXLoadedDiagnosticSetImpl() : CXDiagnosticSetImpl(true), FakeFiles(FO) {} - ~CXLoadedDiagnosticSetImpl() override = default; + ~CXLoadedDiagnosticSetImpl() override {} llvm::BumpPtrAllocator Alloc; Strings Categories; @@ -63,7 +63,7 @@ public: // Cleanup. //===----------------------------------------------------------------------===// -CXLoadedDiagnostic::~CXLoadedDiagnostic() = default; +CXLoadedDiagnostic::~CXLoadedDiagnostic() {} //===----------------------------------------------------------------------===// // Public CXLoadedDiagnostic methods. diff --git a/tools/libclang/Indexing.cpp b/tools/libclang/Indexing.cpp index c8b55e00d5..8a0498497e 100644 --- a/tools/libclang/Indexing.cpp +++ b/tools/libclang/Indexing.cpp @@ -151,7 +151,9 @@ class SessionSkipBodyData { public: SessionSkipBodyData() : Mux(/*recursive=*/false) {} - ~SessionSkipBodyData() = default; + ~SessionSkipBodyData() { + //llvm::errs() << "RegionData: " << Skipped.size() << " - " << Skipped.getMemorySize() << "\n"; + } void copyTo(PPRegionSetTy &Set) { llvm::MutexGuard MG(Mux); diff --git a/unittests/AST/EvaluateAsRValueTest.cpp b/unittests/AST/EvaluateAsRValueTest.cpp index 4530bc95fb..820edbc7c3 100644 --- a/unittests/AST/EvaluateAsRValueTest.cpp +++ b/unittests/AST/EvaluateAsRValueTest.cpp @@ -66,7 +66,7 @@ class EvaluateConstantInitializersAction : public clang::ASTFrontendAction { private: class Consumer : public clang::ASTConsumer { public: - ~Consumer() override = default; + ~Consumer() override {} void HandleTranslationUnit(clang::ASTContext &Ctx) override { VarInfoMap VarInfo; diff --git a/unittests/ASTMatchers/ASTMatchersTest.h b/unittests/ASTMatchers/ASTMatchersTest.h index b1456fd105..9ed7ef66e5 100644 --- a/unittests/ASTMatchers/ASTMatchersTest.h +++ b/unittests/ASTMatchers/ASTMatchersTest.h @@ -26,7 +26,7 @@ using clang::tooling::FileContentMappings; class BoundNodesCallback { public: - virtual ~BoundNodesCallback() = default; + virtual ~BoundNodesCallback() {} virtual bool run(const BoundNodes *BoundNodes) = 0; virtual bool run(const BoundNodes *BoundNodes, ASTContext *Context) = 0; virtual void onEndOfTranslationUnit() {} diff --git a/unittests/ASTMatchers/Dynamic/ParserTest.cpp b/unittests/ASTMatchers/Dynamic/ParserTest.cpp index f181d6d19e..ef66a816aa 100644 --- a/unittests/ASTMatchers/Dynamic/ParserTest.cpp +++ b/unittests/ASTMatchers/Dynamic/ParserTest.cpp @@ -23,7 +23,7 @@ namespace { class MockSema : public Parser::Sema { public: - ~MockSema() override = default; + ~MockSema() override {} uint64_t expectMatcher(StringRef MatcherName) { // Optimizations on the matcher framework make simple matchers like diff --git a/unittests/Tooling/CompilationDatabaseTest.cpp b/unittests/Tooling/CompilationDatabaseTest.cpp index d45871f729..380d86fc56 100644 --- a/unittests/Tooling/CompilationDatabaseTest.cpp +++ b/unittests/Tooling/CompilationDatabaseTest.cpp @@ -173,7 +173,7 @@ TEST(JSONCompilationDatabase, ArgumentsPreferredOverCommand) { } struct FakeComparator : public PathComparator { - ~FakeComparator() override = default; + ~FakeComparator() override {} bool equivalent(StringRef FileA, StringRef FileB) const override { return FileA.equals_lower(FileB); } diff --git a/unittests/Tooling/RewriterTestContext.h b/unittests/Tooling/RewriterTestContext.h index ae593f08a5..eee7ea1873 100644 --- a/unittests/Tooling/RewriterTestContext.h +++ b/unittests/Tooling/RewriterTestContext.h @@ -50,7 +50,7 @@ class RewriterTestContext { OverlayFileSystem->pushOverlay(InMemoryFileSystem); } - ~RewriterTestContext() = default; + ~RewriterTestContext() {} FileID createInMemoryFile(StringRef Name, StringRef Content) { std::unique_ptr<llvm::MemoryBuffer> Source = diff --git a/unittests/Tooling/TestVisitor.h b/unittests/Tooling/TestVisitor.h index a69e18eee1..f4a0039448 100644 --- a/unittests/Tooling/TestVisitor.h +++ b/unittests/Tooling/TestVisitor.h @@ -35,9 +35,9 @@ namespace clang { template <typename T> class TestVisitor : public RecursiveASTVisitor<T> { public: - TestVisitor() = default; + TestVisitor() { } - virtual ~TestVisitor() = default; + virtual ~TestVisitor() { } enum Language { Lang_C, |