diff options
author | Pavel Labath <labath@google.com> | 2018-05-31 09:46:26 +0000 |
---|---|---|
committer | Pavel Labath <labath@google.com> | 2018-05-31 09:46:26 +0000 |
commit | 34cda14b09e0e55f59ce374265207d9af04d27be (patch) | |
tree | 8048944a9b23fa6a48ccfd966b08b2cd467c0bca /lldb/source/Commands/CommandObjectTarget.cpp | |
parent | af3226752b4de38df6b486f5e8949152d1c1daed (diff) | |
download | bcm5719-llvm-34cda14b09e0e55f59ce374265207d9af04d27be.tar.gz bcm5719-llvm-34cda14b09e0e55f59ce374265207d9af04d27be.zip |
Remove append parameter to FindGlobalVariables
Summary:
As discussed in https://bugs.llvm.org/show_bug.cgi?id=37317,
FindGlobalVariables does not properly handle the case where
append=false. As this doesn't seem to be used in the tree, this patch
removes the parameter entirely.
Reviewers: clayborg, jingham, labath
Reviewed By: clayborg
Subscribers: aprantl, lldb-commits, kubamracek, JDevlieghere
Differential Revision: https://reviews.llvm.org/D46885
Patch by Tom Tromey <ttromey@mozilla.com>.
llvm-svn: 333639
Diffstat (limited to 'lldb/source/Commands/CommandObjectTarget.cpp')
-rw-r--r-- | lldb/source/Commands/CommandObjectTarget.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lldb/source/Commands/CommandObjectTarget.cpp b/lldb/source/Commands/CommandObjectTarget.cpp index ee7e0e914a2..b383b55c1fa 100644 --- a/lldb/source/Commands/CommandObjectTarget.cpp +++ b/lldb/source/Commands/CommandObjectTarget.cpp @@ -752,7 +752,7 @@ public: VariableList &variable_list) { Target *target = static_cast<Target *>(baton); if (target) { - return target->GetImages().FindGlobalVariables(ConstString(name), true, + return target->GetImages().FindGlobalVariables(ConstString(name), UINT32_MAX, variable_list); } return 0; @@ -818,8 +818,8 @@ protected: return false; } use_var_name = true; - matches = target->GetImages().FindGlobalVariables( - regex, true, UINT32_MAX, variable_list); + matches = target->GetImages().FindGlobalVariables(regex, UINT32_MAX, + variable_list); } else { Status error(Variable::GetValuesForVariableExpressionPath( arg, m_exe_ctx.GetBestExecutionContextScope(), @@ -947,8 +947,8 @@ protected: llvm::StringRef( ".")); // Any global with at least one character VariableList variable_list; - sc.module_sp->FindGlobalVariables(all_globals_regex, append, - UINT32_MAX, variable_list); + sc.module_sp->FindGlobalVariables(all_globals_regex, UINT32_MAX, + variable_list); DumpGlobalVariableList(m_exe_ctx, sc, variable_list, s); } } |