rjmccall added inline comments.
================
Comment at: clang/lib/Sema/JumpDiagnostics.cpp:362
// it. This makes the second scan not have to walk the AST again.
+RecordJumpScope:
LabelAndGotoScopes[S] = ParentScope;
----------------
I would indent this at the same level as the `case` labels, and group it with
them, too, but yeah, this is what I was thinking. But the helper function
approach looks pretty clean, too. Your choice.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155525/new/
https://reviews.llvm.org/D155525
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
