[PATCH] D130801: [clang-tidy] Add trace code for ASTVisitors

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 29 15:04:00 PDT 2022


njames93 created this revision.
njames93 added reviewers: alexfh, aaron.ballman.
Herald added subscribers: carlosgalvezp, xazax.hun, mgorny.
Herald added a project: All.
njames93 requested review of this revision.
Herald added a project: clang-tools-extra.
Herald added a subscriber: cfe-commits.

This was some code I wrote to help debug crashes when refactoring the SimplifyBooleanExpr check.
I feel like this could help fix other potential crashes in checks that make use of ASTVisitors.
It would also bring the crashing debug info more in line with what we have for matcher based checks.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D130801

Files:
  clang-tools-extra/clang-tidy/readability/SimplifyBooleanExprCheck.cpp
  clang-tools-extra/clang-tidy/utils/CMakeLists.txt
  clang-tools-extra/clang-tidy/utils/VisitorTrace.cpp
  clang-tools-extra/clang-tidy/utils/VisitorTrace.h
  clang-tools-extra/unittests/clang-tidy/CMakeLists.txt
  clang-tools-extra/unittests/clang-tidy/VisitorTraceTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130801.448727.patch
Type: text/x-patch
Size: 10597 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220729/6052ceff/attachment.bin>


More information about the cfe-commits mailing list