Serene
/
include-what-you-use
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove now-redundant InstantiateImplicitMethods calls 

Now that shouldVisitImplicitCode is on, implicit constructors are properly
visited and we don't need to instantiate them explicitly.

No functional change.
This commit is contained in:
Kim Grasman 2020-07-31 18:13:13 +02:00 committed by Kim Gräsman
parent 8774126c67
commit 2d9bd45816
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
iwyu.cc
View File

@ -771,8 +771,6 @@ class BaseAstVisitor : public RecursiveASTVisitor<Derived> {
!IsCXXConstructExprInInitializer(current_ast_node()) &&
!IsCXXConstructExprInNewExpr(current_ast_node());
if (will_call_implicit_destructor_on_leaving_scope) {
// Create the destructor if it hasn't been lazily created yet.
InstantiateImplicitMethods(expr->getConstructor()->getParent());
if (const CXXDestructorDecl* dtor_decl = GetSiblingDestructorFor(expr)) {
if (!this->getDerived().TraverseImplicitDestructorCall(
const_cast<CXXDestructorDecl*>(dtor_decl), GetTypeOf(expr)))
@ -787,7 +785,6 @@ class BaseAstVisitor : public RecursiveASTVisitor<Derived> {
if (CanIgnoreCurrentASTNode()) return true;
// In this case, we *know* we're responsible for destruction as well.
InstantiateImplicitMethods(expr->getConstructor()->getParent());
CXXConstructorDecl* ctor_decl = expr->getConstructor();
CXXDestructorDecl* dtor_decl =
const_cast<CXXDestructorDecl*>(GetSiblingDestructorFor(expr));