[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