TPPPP72 wrote:

> maybe we should remove that FIXME instead.

It can be a solution (⁠・⁠o⁠・⁠)

> On the other hand I don't think the heap allocation is a big deal here

Same here. I don't really think Clang's compile time will see any noticeable 
improvement. As for clangd, it might lead to a slight boost in live code 
analysis.

> do you have [c-t-t](https://github.com/nikic/llvm-compile-time-tracker) 
> statistics against the patch?

Not yet — I don't have the numbers handy right now, but I can generate them if 
necessary.



https://github.com/llvm/llvm-project/pull/218264
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to