https://github.com/Sirraide requested changes to this pull request.

I was following this on Discourse, so I thought I’d take a look; overall, this 
seems like a very interesting addition to Clang to me.

This is by no means a complete review; this pr is pretty big, so looking at 
everything in detail would take me a lot longer than just an hour or two, so 
I’ve just pointed out things that stood out to me. I’m also not particularly 
familiar with the analysis-based-warnings part of Clang, which is where most of 
the code for this pr seems to live, so I won’t be of much help there.

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

Reply via email to