This fixes a regression that led to size() not being available in clangd
when completing 'deque().^' and using libc++.
Details
Details
- Reviewers
sammccall - Commits
- rG8613e90ba71d: [CodeComplete] Ensure object is the same in compareOverloads()
rC373710: [CodeComplete] Ensure object is the same in compareOverloads()
rCTE373710: [CodeComplete] Ensure object is the same in compareOverloads()
rL373710: [CodeComplete] Ensure object is the same in compareOverloads()
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp | ||
---|---|---|
2565 ↗ | (On Diff #222821) | could you give the three methods different return types, so it's clearer which one is being selected? |
clang/lib/Sema/SemaCodeComplete.cpp | ||
1188 ↗ | (On Diff #222821) | This comment is a little confusing: the very next line takes object type into account :-) Maybe: // Base/derived shadowing is handled elsewhere. |
Comment Actions
- Address comments
clang/lib/Sema/SemaCodeComplete.cpp | ||
---|---|---|
1188 ↗ | (On Diff #222821) | I also did not like it, thanks for the great suggestion. |