HsiangKai added a comment. In D112102#3075400 <https://reviews.llvm.org/D112102#3075400>, @craig.topper wrote:
> In D112102#3074656 <https://reviews.llvm.org/D112102#3074656>, @frasercrmck > wrote: > >> Minor typo in the description: `differnet` >> >> Does this help with compile times, binary sizes, etc? > > This reduces the clang binary size by 2.34 megabytes according to my local > release+asserts build. > > I think there might be a small decrease in lit test time. If not, I don't see > any reason it would increase with this patch. The header file size does > decrease. My fault. This patch decreases the binary size. The testing time increases due to D112020 <https://reviews.llvm.org/D112020>. This patch decreases the header file size indeed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112102/new/ https://reviews.llvm.org/D112102 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
