[PATCH] D146717: [clangd] Handle the C++2b elifdef and elindef PP structure in CollectMainFileMacros.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 29 00:42:49 PDT 2023
hokein added inline comments.
================
Comment at: clang-tools-extra/clangd/CollectMacros.h:69
const MacroDefinition &MD) override;
+ using PPCallbacks::Elifdef;
+ using PPCallbacks::Elifndef;
----------------
kadircet wrote:
> why do we have these using decls?
This is to suppress the -Woverloaded-virtual diagnostic from gcc (https://godbolt.org/z/9b76fYbjx) -- PPCallback has two overloads of `Elifdefs`, here we only override one.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146717/new/
https://reviews.llvm.org/D146717
More information about the cfe-commits
mailing list