https://github.com/NagyDonat commented:

Thanks for your patience in this review! I'm happy to see the additional 
simplifications coming from just wrapping the over-complicated symbol instead 
of storing its parts separately. (I see that it's a bit inelegant to construct 
the very thing that we're trying to avoid, but this is the pragmatic choice...)

I don't expect any performance issues (the "frozen" complex symbols shouldn't 
be problematic if they aren't traversed), but performance testing is never 
completely useless.

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

Reply via email to