diff options
| author | Zachary Turner <zturner@google.com> | 2019-01-14 22:41:21 +0000 |
|---|---|---|
| committer | Zachary Turner <zturner@google.com> | 2019-01-14 22:41:21 +0000 |
| commit | 576495e67b3c7d3d2f98e7af312cb03e8f63f1cd (patch) | |
| tree | d460cc909087c236c8dd77857296b903580f8505 /lldb/tools/lldb-test/lldb-test.cpp | |
| parent | c0a246afbe03b8f1d52e06267e9c2a5f2c4521ff (diff) | |
| download | bcm5719-llvm-576495e67b3c7d3d2f98e7af312cb03e8f63f1cd.tar.gz bcm5719-llvm-576495e67b3c7d3d2f98e7af312cb03e8f63f1cd.zip | |
[SymbolFile] Remove SymbolContext parameter from FindTypes.
This parameter was only ever used with the Module set, and
since a SymbolFile is tied to a module, the parameter turns
out to be entirely unnecessary. Furthermore, it doesn't make
a lot of sense to ask a caller to ask SymbolFile which is tied
to Module X to find types for Module Y, but that possibility
was open with the previous interface. By removing this
parameter from the API, it makes it harder to use incorrectly
as well as easier for an implementor to understand what it
needs to do.
llvm-svn: 351133
Diffstat (limited to 'lldb/tools/lldb-test/lldb-test.cpp')
| -rw-r--r-- | lldb/tools/lldb-test/lldb-test.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lldb/tools/lldb-test/lldb-test.cpp b/lldb/tools/lldb-test/lldb-test.cpp index 92c178098a1..f7bfc487c0c 100644 --- a/lldb/tools/lldb-test/lldb-test.cpp +++ b/lldb/tools/lldb-test/lldb-test.cpp @@ -459,10 +459,9 @@ Error opts::symbols::findTypes(lldb_private::Module &Module) { CompilerDeclContext *ContextPtr = ContextOr->IsValid() ? &*ContextOr : nullptr; - SymbolContext SC; DenseSet<SymbolFile *> SearchedFiles; TypeMap Map; - Vendor.FindTypes(SC, ConstString(Name), ContextPtr, true, UINT32_MAX, + Vendor.FindTypes(ConstString(Name), ContextPtr, true, UINT32_MAX, SearchedFiles, Map); outs() << formatv("Found {0} types:\n", Map.GetSize()); |

