Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Nice!
clang-tools-extra/clangd/XRefs.cpp | ||
---|---|---|
194 | Not thrilled about reusing this after previous patches disentangled it - it works well for macros but if (for example) you have a templated operator<< I think the feature you're adding probably won't work. That said, I don't see a simple alternative, and it's unlikely to be a big deal, so I think this is OK. | |
240 | Be aware that this is going to break "go to definition" toggling between def and decl. I think that's fine. | |
240 | (I think you probably want the macro-arg-expanded location here) |
Address comment
clang-tools-extra/clangd/XRefs.cpp | ||
---|---|---|
240 | For what will this break toggling? Aas far as I'm aware, partial and explicit specializations of templates do not have separate declarations. |
clang-tools-extra/clangd/XRefs.cpp | ||
---|---|---|
240 | I stand corrected: I checked, and it seems like specializations can indeed have forward declarations. I expect they are relatively rare, though, and it's probably fine to break toggling for them, like you said. |
Not thrilled about reusing this after previous patches disentangled it - it works well for macros but if (for example) you have a templated operator<< I think the feature you're adding probably won't work.
That said, I don't see a simple alternative, and it's unlikely to be a big deal, so I think this is OK.