abhina-sree wrote:

> LGTM, but I would also consider dropping the "using namespace llvm" instead. 
> This seems a bit unusual for a file in clang/lib/Analysis.

Thanks! I've implemented your suggested solution instead

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

Reply via email to