iains added a comment. FAOD: I think this should not be about "who's patches" - but about the compilation model and command lines that we want to end up with.
I am concerned that with the current scheme (with or without the change in this review) we could end up with three process launches per modular file: - one to determine the deps - one to produce the BMI - one to produce the object At least with GCC (and a patch series like the one I drafted) we can reduce that to 2 With the interfaces mentioned in P1184 <https://reviews.llvm.org/P1184>, it is plausible to make that 1 (at the expense of potentially many stalled builds - but stalled early in compilation, so hopefully with the smallest memory footprint we could achieve). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134267/new/ https://reviews.llvm.org/D134267 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits