[clang-tools-extra] [clangd] Navigate go-to-definition through forwarding wrappers to the constructor (PR #199480)

Yuli Fiterman via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 25 20:17:35 PDT 2026


fitermay wrote:

> Alright, so I have finally gotten around to give this a full review. Overall I think this looks good, the only thing I am a bit unsure about is you adding 6 new tests. Not saying I dislike you testing well, but I am wondering more about combining them into one or two tests, most of them have a large overlap in code and could easily be extended to contain multiple of the others (you can name code points too to have multiple). Would probably also be nice for speed.

Got it. Consolidated the tests into one

https://github.com/llvm/llvm-project/pull/199480


More information about the cfe-commits mailing list