ChuanqiXu added a comment. Now I think the feature may be important for the performance of modules. And I feel we should work on the ASTWriter side instead of ASTReader side. Since the size of BMIs is a problem now also it shows that it is not free to load the large BMIs. So while it is semantical correct to work on the reader side, it is better for the performance to work on the writer side.
I'd like to finish the idea. And for the current patch, I'd like to refactor it a little bit: 1. Test it by unittest instead of by matching the dump result. 2. Remove the Serialization part. So it will be a NFC patch. 3. Some other trivial polishment. Of course, I'll still mark you as the author. How do you feel about this? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D126694/new/ https://reviews.llvm.org/D126694 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
