[PATCH] D112137: [clangd] Only publish preamble after rebuilds

2021-10-20 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. kadircet marked an inline comment as done. Closed by commit rGc959da9ef344: [clangd] Only publish preamble after rebuilds (authored by kadircet). Changed prior to commit: https://reviews.llvm.org/D112137?vs=380931&id=3809

[PATCH] D112137: [clangd] Only publish preamble after rebuilds

2021-10-20 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/TUScheduler.cpp:904 const ParseInputs &Inputs = Req.Inputs; + bool RebuiltPreamble = true; I think it wo

[PATCH] D112137: [clangd] Only publish preamble after rebuilds

2021-10-20 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: usaxena95, arphaman, javed.absar. kadircet requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. Don't invoke parsi