AaronBallman wrote:

First off: thank you for the patch and trying to improve Clang!

> Is it necessary to have this feature? Clangd already [supports 
> it](https://clangd.llvm.org/config.html#inlayhints).

Also, Clang itself already has the ability to dump information about the AST: 
https://godbolt.org/z/aEM53YKq9
```
`-VarDecl 0x1be884d8 <<source>:1:1, col:10> col:6 x 'int' cinit
  `-IntegerLiteral 0x1be88588 <col:10> 'int' 12
```
(and has ways to filter AST dumps so there's less information about unrelated 
nodes), so I also question whether this functionality is necessary under 
another flag. I think the existing facilities already suffice, but maybe I'm 
missing something?

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

Reply via email to