https://github.com/cor3ntin commented:

This is a neat change. I only did a cursory review for now but I like the 
direct and did not spot anything jaring. We do need to be a bit more careful 
about the AST matcher changes though

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

Reply via email to