zixuw added inline comments.
================
Comment at: clang/lib/SymbolGraph/ExtractAPIConsumer.cpp:197
+ void recordEnumConstants(EnumRecord *EnumRecord,
+ const EnumDecl::enumerator_range Constants) {
----------------
dang wrote:
> Should this be static or in an anonymous namespace?
The whole `ExtractAPIVisitor` class is already in an anonymous namespace right?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121873/new/
https://reviews.llvm.org/D121873
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
