[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

[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-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