This is an archive of the discontinued LLVM Phabricator instance.

[CodeComplete] Ensure object is the same in compareOverloads()
ClosedPublic

Authored by ilya-biryukov on Oct 2 2019, 6:45 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ilya-biryukov created this revision.Oct 2 2019, 6:45 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 2 2019, 6:45 AM
ilya-biryukov edited the summary of this revision. (Show Details)Oct 2 2019, 6:58 AM
sammccall added inline comments.Oct 2 2019, 7:05 AM
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.

sammccall accepted this revision.Oct 2 2019, 7:05 AM

(oops, meant to accept with comments)

This revision is now accepted and ready to land.Oct 2 2019, 7:05 AM
ilya-biryukov marked 3 inline comments as done.
  • Address comments
clang/lib/Sema/SemaCodeComplete.cpp
1188 ↗(On Diff #222821)

I also did not like it, thanks for the great suggestion.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2019, 1:12 AM