https://github.com/OCHyams commented:
Hi, sorry it took me some time to get to this. It's already got approval and I'm not not overly familiar with some of this, but FWIW I think it SGTM too. I've got some inline questions that are to help me understand the change a bit more, but I don't think should block you merging this. https://github.com/llvm/llvm-project/pull/153883 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
