[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers
Daniel Grumberg via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 23 07:03:29 PDT 2022
dang marked an inline comment as done.
dang added inline comments.
================
Comment at: clang/include/clang/SymbolGraph/FrontendActions.h:35
+ /// This is called before executing the action on any inputs. This merges all
+ /// the provided headers into a single header for processing.
+ bool PrepareToExecuteAction(CompilerInstance &CI) override;
----------------
ributzka wrote:
> This comment confused me a bit. By looking at the code further down, it only adds include statements for each header, which is different from merging all headers into a single header/buffer.
You are right in that is a bit confusing. I have reworded this to reflect what is actually happening more accurately.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122175/new/
https://reviews.llvm.org/D122175
More information about the cfe-commits
mailing list