ilya-biryukov added a comment.
In https://reviews.llvm.org/D38617#892092, @bkramer wrote:
> A testcase would be nice, but this can go in to unblock things.
Thanks for reviewing this!
Added a test case to clangd in https://reviews.llvm.org/rL315213.
Repository:
rL LLVM
https://reviews.llvm.
This revision was automatically updated to reflect the committed changes.
Closed by commit rL315212: Set PreprocessorOpts.GeneratePreamble=true in
PrecompiledPreamble. (authored by ibiryukov).
Repository:
rL LLVM
https://reviews.llvm.org/D38617
Files:
cfe/trunk/lib/Frontend/ASTUnit.cpp
cf
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
A testcase would be nice, but this can go in to unblock things.
https://reviews.llvm.org/D38617
___
cfe-commits mailing list
cfe-commits@lists.
ilya-biryukov added reviewers: bkramer, klimek.
ilya-biryukov added a comment.
Adding more reviewers, in case someone will have time to look at that :-)
https://reviews.llvm.org/D38617
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://l
ilya-biryukov added a comment.
This commit will unbreak clangd from all the same issues that were fixed by
conditional stack commits.
https://reviews.llvm.org/D38617
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-
ilya-biryukov created this revision.
It was previsouly set only in ASTUnit, but it should be set for all client of
PrecompiledPreamble.
https://reviews.llvm.org/D38617
Files:
lib/Frontend/ASTUnit.cpp
lib/Frontend/PrecompiledPreamble.cpp
Index: lib/Frontend/PrecompiledPreamble.cpp
