jonathonpenix wrote:

Also few misc. notes:

1. Please let me know if this needs to be discussed elsewhere and I'd be happy 
to do so! I was originally thinking to ping https://reviews.llvm.org/D130481 / 
https://reviews.llvm.org/D129178 to check if this made sense, but they are 
read-only now IIUC.
2. I'm also not certain if there were other concerns about making GlobalMerge 
more easily accessible to users beyond what was mentioned in 
https://reviews.llvm.org/D129178 (if there are correctness concerns, etc. that 
I missed). My reading of the regressions mentioned was that they were code 
size/performance related (rather than correctness) but I might be 
misunderstanding/have missed something.

Thanks!

https://github.com/llvm/llvm-project/pull/79372
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to