HighCommander4 wrote:

Some high-level feedback:

 1. Hovers contain a variety of types of content (types, values, definitions, 
initializers, documentation, etc.). This limit applies specifically to macro 
expansions; therefore, it should have a more specific name, e.g. "limit hover 
macro expansion" or similar.
 2. Kadir mentioned making this a config option in [this 
comment](https://github.com/llvm/llvm-project/issues/153355#issuecomment-3187831592),
 but the current patch is making it a command-line option, is that intentional?

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

Reply via email to