[clang] [clang][AST] Inline StmtVisitor fallback methods (PR #203125)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 6 07:10:44 PDT 2026


================
@@ -115,13 +115,16 @@ class StmtVisitorBase {
 
   // If the implementation chooses not to implement a certain visit method, fall
   // back on VisitExpr or whatever else is the superclass.
+  // clang-format off
----------------
erichkeane wrote:

In at least a few of those it is much better, and I'd rather have those (particularly `STMT` nad `BINOP_FALLBACK`.  But what it did to the lists is obviously horrifying.  Could you tighten the clang-format off to be more targetted to only the ones that are particularly awful?  The project has a general 'disagree-and-commit' to whatever clang-format gives us unless particularly horrifying.

https://github.com/llvm/llvm-project/pull/203125


More information about the cfe-commits mailing list