[PATCH] D126077: Fix stack crash in classIsDerivedFrom triggered by clang-tidy

2022-05-24 Thread Anton Fedorov via Phabricator via cfe-commits
datacompboy added a comment. In D126077#3530189 , @njames93 wrote: > Do you know of any other instances where this issue could surface? I didn't found any other cases where isDerivedFrom used directly, only in couple of internal checks, none of which ar

[PATCH] D126077: Fix stack crash in classIsDerivedFrom triggered by clang-tidy

2022-05-23 Thread Anton Fedorov via Phabricator via cfe-commits
datacompboy updated this revision to Diff 431339. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126077/new/ https://reviews.llvm.org/D126077 Files: clang/lib/ASTMatchers/ASTMatchFinder.cpp clang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp

[PATCH] D126077: Fix stack crash in classIsDerivedFrom triggered by clang-tidy

2022-05-22 Thread Nathan James via Phabricator via cfe-commits
njames93 edited reviewers, added: aaron.ballman, njames93, alexfh, LegalizeAdulthood; removed: bixia, aartbik. njames93 added a comment. Do you know of any other instances where this issue could surface? Please can you add a line to the `Improved Checks` section in `clang-tools-extra/docs/Relea

[PATCH] D126077: Fix stack crash in classIsDerivedFrom triggered by clang-tidy

2022-05-20 Thread Anton Fedorov via Phabricator via cfe-commits
datacompboy created this revision. datacompboy added reviewers: bixia, aartbik. Herald added a project: All. datacompboy requested review of this revision. Herald added projects: clang, clang-tools-extra. Herald added a subscriber: cfe-commits. Protect classIsDerivedFrom from run-out-of-stack cras