zyn0217 wrote: I think we should go with the status quo given that there are a large number uses of the function - maybe we should remove that FIXME instead.
On the other hand I don't think the heap allocation is a big deal here, do you have [c-t-t](https://github.com/nikic/llvm-compile-time-tracker) statistics against the patch? https://github.com/llvm/llvm-project/pull/218264 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
