[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. dang marked an inline comment as done. Closed by commit rGf833aab0d0bf: [clang][extract-api] Enable processing of multiple headers (authored by dang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-23 Thread Juergen Ributzka via Phabricator via cfe-commits
ributzka accepted this revision. ributzka added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122175/new/ https://reviews.llvm.org/D122175 ___ cfe-commits mailing list

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
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. +

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang updated this revision to Diff 417596. dang added a comment. Clarify doc comment for ExtractAPIAction::PrepareToExecuteAction Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122175/new/ https://reviews.llvm.org/D122175 Files:

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang updated this revision to Diff 417593. dang added a comment. Rebase on top of latest changes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122175/new/ https://reviews.llvm.org/D122175 Files: clang/include/clang/ExtractAPI/FrontendActions.h

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-22 Thread Juergen Ributzka via Phabricator via cfe-commits
ributzka 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

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-21 Thread Daniel Grumberg via Phabricator via cfe-commits
dang added a comment. This is up as a "draft" right now. I am waiting on https://reviews.llvm.org/D122160 to land before doing a final rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122175/new/ https://reviews.llvm.org/D122175

[PATCH] D122175: [clang][extract-api] Enable processing of multiple headers

2022-03-21 Thread Daniel Grumberg via Phabricator via cfe-commits
dang created this revision. dang added reviewers: ributzka, zixuw, QuietMisdreavus. Herald added a project: All. dang requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Before actually executing the ExtractAPIAction, clear the