https://github.com/jansvoboda11 approved this pull request.
LGTM with one nit suggestion. There seems to be a build failure in CI due to revert of @Bigcheese's commit, so that probably needs to be taken care of before merging. I'd also like to see the downstream PR before merging this one just so we know things integrate nicely with the C API. https://github.com/llvm/llvm-project/pull/144105 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits