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