[Lldb-commits] [PATCH] D15312: Fix scope-based lookup when more than one function is found.

[Dawn Perchik via lldb-commits]

dawn updated this revision to Diff 42385.
dawn added a comment.

This version of the patch is the API "generic" version of the patch.  Please choose to review which ever is preferred, and I will submit another patch based on that version if needed.

BTW Greg, thanks for the suggestion of multimap - it definitely simplified things!  And if you're wondering why we want to keep a set of functions for each type in FindExternalVisibleDecls, it's because overload resolution hasn't chosen a function yet, so we must keep all possible candidates in our set.


Repository:
  rL LLVM

http://reviews.llvm.org/D15312

Files:
  include/lldb/Symbol/ClangASTContext.h
  include/lldb/Symbol/GoASTContext.h
  include/lldb/Symbol/TypeSystem.h
  packages/Python/lldbsuite/test/lang/cpp/namespace/Makefile
  packages/Python/lldbsuite/test/lang/cpp/namespace/main.cpp
  packages/Python/lldbsuite/test/lang/cpp/namespace/ns.cpp
  packages/Python/lldbsuite/test/lang/cpp/namespace/ns.h
  packages/Python/lldbsuite/test/lang/cpp/namespace/ns2.cpp
  packages/Python/lldbsuite/test/lang/cpp/namespace/ns3.cpp
  source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp
  source/Symbol/ClangASTContext.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15312.42385.patch
Type: text/x-patch
Size: 22353 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151210/5adafd29/attachment-0001.bin>