hans added a comment. In D74846#1895319 <https://reviews.llvm.org/D74846#1895319>, @aganea wrote:
> Thanks @hans! Nevertheless I think this is a good change and we should push > it for 11.0. I was indeed seeing better timings on our end with what @llunak > was proposing, we are heavy users of PCH headers (and migrating to modules is > not trivial in our case, lots of code branching & integrations) Right, I have no objections against this re-landing on master once the problem in PR44953 is understood and fixed. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74846/new/ https://reviews.llvm.org/D74846 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits