[PATCH] D158646: [clang-tools-extra][ExtractAPI] create clang-symbolgraph-merger

Arsenic added a comment. In D158646#4639599 , @dang wrote: > As per https://discourse.llvm.org/t/pull-request-migration-schedule/71595 we > should move this review to GitHub to make sure we don't lose track of it. The patch has been moved to GitHub (

[PATCH] D157810: [clang][ExtractAPI] Create extractapi::RecordLocation

Arsenic updated this revision to Diff 553730. Arsenic added a comment. Fix minor typo is comments - Move creation of DocComment in it's own seperate function - Update the Underlying DataType used by RecordLocation - Update the patch to account for new C++ APIRecords Repository: rG LLVM

[PATCH] D158646: [clang-tools-extra][ExtractAPI] create clang-symbolgraph-merger

Arsenic created this revision. Arsenic added a reviewer: dang. Arsenic added projects: All, clang-tools-extra. Herald added a reviewer: ributzka. Arsenic requested review of this revision. Create a clang tool to merge all the JSON symbolgraph emited by --emit-symbol-graph or -extract-api options

[PATCH] D157810: [clang][ExtractAPI] Create extractapi::RecordLocation

Arsenic updated this revision to Diff 552794. Arsenic added a comment. - Create function to directly get DocComments from Decl CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157810/new/ https://reviews.llvm.org/D157810 Files: clang/include/clang/ExtractAPI/API.h

[PATCH] D157810: [clang][ExtractAPI] Create extractapi::RecordLocation

Arsenic updated this revision to Diff 549691. Arsenic added a comment. Fix minor typo in code comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D157810/new/ https://reviews.llvm.org/D157810 Files: clang/include/clang/ExtractAPI/API.h

[PATCH] D157810: [clang][ExtractAPI] Create extractapi::RecordLocation

Arsenic created this revision. Arsenic added a reviewer: dang. Herald added a reviewer: ributzka. Herald added a project: All. Arsenic requested review of this revision. Herald added subscribers: cfe-commits, wangpc. Herald added a project: clang. Create and use extractapi::RecordLocation instead

[PATCH] D152356: [clang][ExtractAPI] Add --emit-symbol-graph option

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8e9145e43142: [clang][ExtractAPI] Add --emit-symbol-graph option (authored by Arsenic). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D152356: [clang][ExtractAPI] Add --emit-symbol-graph option

Arsenic updated this revision to Diff 535630. Arsenic added a comment. Changes majorly include fixing issues pointed out by review comments : - Remove WrappingExtractAPIAction::prepareToExecuteAction() - Move knownInputFiles from Base action to ExtractAPIFrontendAction - Fix naming of

[PATCH] D152356: [clang][ExtractAPI] Add --emit-symbol-graph option

Arsenic created this revision. Arsenic added a reviewer: dang. Herald added a reviewer: ributzka. Herald added a project: All. Arsenic updated this revision to Diff 529841. Arsenic added a comment. Arsenic marked 5 inline comments as done. Arsenic updated this revision to Diff 532145. Arsenic

[PATCH] D145869: [clang][ExtractAPI] Add multiple file support to --extract-api-ignores

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG58825d2cf947: [clang][ExtractAPI] Add multiple file support to --extract-api-ignores (authored by Arsenic). Repository: rG LLVM Github Monorepo

[PATCH] D145869: [clang][ExtractAPI] Add multiple file support to --extract-api-ignores

Arsenic updated this revision to Diff 504594. Arsenic added a comment. fix test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145869/new/ https://reviews.llvm.org/D145869 Files: clang/include/clang/Driver/Options.td

[PATCH] D145869: [clang][ExtractAPI] Add multiple file support to --extract-api-ignores

Arsenic updated this revision to Diff 504590. Arsenic added a comment. Update clang/test/ExtractAPI/ignored-symbols-multifile.c : Shuffle the order of ignored symbols in test to check the sorting behaviour of APIIgnoresList Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D145869: [clang][ExtractAPI] Add multiple file support to --extract-api-ignores

Arsenic created this revision. Herald added a reviewer: ributzka. Herald added a project: All. Arsenic requested review of this revision. Herald added a reviewer: dang. Herald added a project: clang. Herald added a subscriber: cfe-commits. - Modify -extract-api-ignores command line option to

[PATCH] D144964: [clang][ExtractAPI] Handle platform specific unavailability correctly

Arsenic abandoned this revision. Arsenic added a comment. duplicate of https://reviews.llvm.org/D144940 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144964/new/ https://reviews.llvm.org/D144964 ___

[PATCH] D144940: [clang][ExtractAPI] Handle platform specific unavailability correctly

Arsenic updated this revision to Diff 501148. Arsenic added a comment. Add test to check platform specific unavailability The update also remove the useless changes that were introduced by using clang format on the entire file instead of current commit. Repository: rG LLVM Github Monorepo

[PATCH] D144964: [clang][ExtractAPI] Handle platform specific unavailability correctly

Arsenic created this revision. Herald added a reviewer: ributzka. Herald added a project: All. Arsenic requested review of this revision. Herald added a reviewer: dang. Herald added a project: clang. Herald added a subscriber: cfe-commits. This Patch gives ExtractAPI the ability to emit correct

[PATCH] D144940: [clang][ExtractAPI] Handle platform specific unavailability correctly

Arsenic added a comment. In D144940#4158020 , @dang wrote: > Nice! glad to see this getting fixed. You should add a lit test to ensure we > don't regress this behavior in the future. I see a test checking for availability attribute already exists (

[PATCH] D144940: [clang][ExtractAPI] Handle platform specific unavailability correctly

Arsenic created this revision. Herald added a reviewer: ributzka. Herald added a project: All. Arsenic edited the summary of this revision. Arsenic added reviewers: dang, zixuw. Arsenic published this revision for review. Arsenic added inline comments. Herald added a project: clang. Herald added a