[PATCH] D146279: [clangd] Extend CollectMainFileMacros.

2023-03-23 Thread Haojian Wu via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG002c4b7b955b: [clangd] Extend CollectMainFileMacros. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D146279: [clangd] Extend CollectMainFileMacros.

2023-03-23 Thread Haojian Wu via Phabricator via cfe-commits
hokein added inline comments. Comment at: clang-tools-extra/clangd/CollectMacros.h:48 public: - explicit CollectMainFileMacros(const SourceManager , MainFileMacros ) - : SM(SM), Out(Out) {} + explicit CollectMainFileMacros(const SourceManager , +

[PATCH] D146279: [clangd] Extend CollectMainFileMacros.

2023-03-23 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 507679. hokein marked an inline comment as done. hokein added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146279/new/ https://reviews.llvm.org/D146279 Files:

[PATCH] D146279: [clangd] Extend CollectMainFileMacros.

2023-03-20 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks! Comment at: clang-tools-extra/clangd/CollectMacros.h:48 public: - explicit CollectMainFileMacros(const SourceManager , MainFileMacros ) - : SM(SM),

[PATCH] D146279: [clangd] Extend CollectMainFileMacros.

2023-03-17 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: kadircet. Herald added a subscriber: arphaman. Herald added a project: All. hokein requested review of this revision. Herald added subscribers: MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. Extend the existing