HighCommander4 wrote: Given that we haven't heard objections, I think we should proceed with using resolved paths in `IncludeInserter` as well. It would probably be best to do that before the clangd 21 release, so that we don't ship a release where two different features are interpreting the config keys in different ways.
@Harald-R, are you interested in writing a patch to use resolved paths in `IncludeInserter`? I'm happy to provide more detailed code pointers if you'd like. https://github.com/llvm/llvm-project/pull/140594 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
