https://github.com/jmorse commented:

This looks great, thanks for putting this all together, one nit inline,

Is this intended to be cherry-picked into the release branch or not? Seeing how 
the original motivation was to ease downstream compatibilities, I'd assumed 
there'd be the need for something to be cherry-picked, and this might be too 
invasive at this stage to be moved across. I don't have a full understanding of 
the use case though.

@OCHyams I believe you did the C API changes, are there any additional concerns 
in this area?

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

Reply via email to