Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 38792 Build 38791: arc lint + arc unit
Event Timeline
clang-tools-extra/clangd/FindTarget.cpp | ||
---|---|---|
589 | This comment is a bit unclear, I'll have to change it. |
clang-tools-extra/clangd/FindTarget.cpp | ||
---|---|---|
569 | ... and we need it because, template template decls are visited through it ? | |
580 | I suppose these corresponds to non-type template paramters, don't they? (which should make it similar to integral case) | |
631 | can we keep dlog ? | |
clang-tools-extra/clangd/unittests/FindTargetTests.cpp | ||
775 | can you also name the template param, add a reference in wrapper and check for it ? |
clang-tools-extra/clangd/FindTarget.cpp | ||
---|---|---|
569 | Yes, exactly. Amended this to the comment. | |
580 | Yes, e.g. see the wrapper<func> example from the tests. IIUC, Declaration can only happen in template arguments outside TemplateArgumentLoc. | |
631 | I've initially removed it to avoid passing DynTypedNode to reportReference, but I also agree this dlog is useful. So making the function signature a bit clunky for the sake of debug logging |
- Restore dlog()
- Check reference to function pointer non-type template parameters in tests
- Extend comment of TraverseTemplateArgumentLoc
... and we need it because, template template decls are visited through it ?