[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