[PATCH] D89987: [analyzer] [NFC] Rename SymbolRef to SymExprRef
Artem Dergachev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 23 10:29:06 PDT 2020
NoQ added a comment.
Honestly i'd rather eliminate `SymExpr` and go with `Symbol` everywhere. It's an overloaded term but appending "Expr" to it doesn't really make it significantly less overloaded. We're also properly namespaced so there wouldn't be any linking issues with the rest of LLVM. "Symbol" is catchy and it's one of the most important concepts to grasp in the static analyzer, i'd love to keep it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89987/new/
https://reviews.llvm.org/D89987
More information about the cfe-commits
mailing list