https://github.com/HighCommander4 commented:
Ok, had another look through the patch and it generally looks quite good. Just have a few minor comments. And one final request before merging: could we double check the index size numbers reported in [this comment](https://github.com/llvm/llvm-project/pull/177273#issuecomment-3786398323)? The size increase there is almost suspiciously low, and I'd just like to verify that it wasn't due to some issue with an earlier version of the patch. https://github.com/llvm/llvm-project/pull/177273 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
