[PATCH] D136455: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph

This revision was automatically updated to reflect the committed changes. Closed by commit rG671709f0e7d4: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph (authored by dang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D13645

[PATCH] D136455: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph

dang added a comment. However, I am not sure it's worthwhile and we might run into edge cases doing it this way with explicitly ignored symbols. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136455/new/ https://reviews.llvm.org/D136455 __

[PATCH] D136455: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph

dang added a comment. In D136455#3879849 , @zixuw wrote: > Is it easy to/worth checking if the target is actually outside of the current > module to keep the output smaller and concise? I think it can be done, since we ignore symbols that don't come fro

[PATCH] D136455: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph

zixuw added a comment. Is it easy to/worth checking if the target is actually outside of the current module to keep the output smaller and concise? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136455/new/ https://reviews.llvm.org/D136455 ___

[PATCH] D136455: [clang][ExtractAPI] Add targetFallback to relationships in symbol graph

dang created this revision. dang added reviewers: zixuw, QuietMisdreavus, ributzka. Herald added a subscriber: yaxunl. Herald added a project: All. dang requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Adds a 'targetFallback' field to relati