zyn0217 wrote:

nit: Please rephrase the commit message before you merge this patch. A large 
piece of AST involving memory addresses is really hard to decipher.

In addition, it would be nice if you can simplify the test case further - it's 
super unclear why a bunch of templates are there outside of the 
`SourceLocation::Current()` - probably one or two is sufficient?

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

Reply via email to