iitianpushkar wrote:

> The logic looks solid, though since I am not very familiar with this part of 
> code I think we might want some confirmation from clang maintainers. Also, I 
> am not sure how to add tests for new API?

Thanks, that's understandable.

Well,I could not find any existing dedicated test for `AnyCall`. But, if a 
dedicated test for this API is preferred, I can add a small 
`clang/unittests/Analysis/AnyCallTest.cpp`.

But, for now, i will wait for suggestions from @Xazax-hun . 

https://github.com/llvm/llvm-project/pull/212934
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to