[PATCH] D155506: [clang][JumpDiagnostics] use StmtClass rather than dyn_cast chain NFC
Dmitri Gribenko via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 17 16:16:34 PDT 2023
gribozavr2 added a comment.
I'm not sure it is actually an anti-pattern. `dyn_cast` will transparently handle subclasses, should any be added in the future, but a switch wouldn't.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155506/new/
https://reviews.llvm.org/D155506
More information about the cfe-commits
mailing list