[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG5ef2ec7e4e12: [clang][extract-api] Suppprt for the module name property in SymbolGraph (authored by dang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang updated this revision to Diff 417572. dang added a comment. Accidentally re-added some of the old files in SymbolGraph/ during the rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122141/new/ https://reviews.llvm.org/D122141 Files:

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang updated this revision to Diff 417570. dang added a comment. Missed some stuff in the previous rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122141/new/ https://reviews.llvm.org/D122141 Files: clang/include/clang/Driver/Options.td

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang updated this revision to Diff 417551. dang added a comment. Rebasing on top of latest main which includes https://reviews.llvm.org/rG89f6b26f1beb2c1344f5cfeb34e405128544c76b Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122141/new/

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-23 Thread Daniel Grumberg via Phabricator via cfe-commits
dang added a comment. In D122141#3400578 , @zixuw wrote: > LGTM. Need to rebase. Do you think `ProductName` should be a general property > of `APISerializer` or something specific to the `SymbolGraphSerializer`? Good question! I think I am going to

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-22 Thread Zixu Wang via Phabricator via cfe-commits
zixuw added a comment. LGTM. Need to rebase. Do you think `ProductName` should be a general property of `APISerializer` or something specific to the `SymbolGraphSerializer`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122141/new/

[PATCH] D122141: [clang][extract-api] Suppprt for the module name property in SymbolGraph

2022-03-21 Thread Daniel Grumberg via Phabricator via cfe-commits
dang created this revision. dang added reviewers: zixuw, ributzka, QuietMisdreavus. Herald added a project: All. dang requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Adds `--product-name=` flag to the clang driver. This gets forwarded to