[PATCH] D153330: [include-cleaner] Ignore the ParmVarDecl itself in WalkAST.cpp

2023-06-20 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG149f309f50cd: [include-cleaner] Ignore the ParmVarDecl itself in WalkAST.cpp (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D153330?vs=532863&id=532870#toc Repository: rG LLV

[PATCH] D153330: [include-cleaner] Ignore the ParmVarDecl itself in WalkAST.cpp

2023-06-20 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks! Comment at: clang-tools-extra/include-cleaner/lib/WalkAST.cpp:219 +// as they don't contribute to the main-file #include. +if (isa(VD)) + return true

[PATCH] D153330: [include-cleaner] Ignore the ParmVarDecl itself in WalkAST.cpp

2023-06-20 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kadircet. Herald added a subscriber: kristof.beyls. Herald added a project: All. hokein requested review of this revision. Herald added a project: clang-tools-extra. This will fix a false positive where a ParamVarDecl happend to be the same na