diff options
author | Siva Chandra <sivachandra@google.com> | 2016-02-04 20:08:40 +0000 |
---|---|---|
committer | Siva Chandra <sivachandra@google.com> | 2016-02-04 20:08:40 +0000 |
commit | 77b326f39d8b33615a57d41d868ae0bbcddd5bf4 (patch) | |
tree | f4061ee556435ffafe8aae789179a7f679999b94 /lldb/source/Expression/ExpressionSourceCode.cpp | |
parent | 6ecf633f9964eec3eecbb53a9ac03d251124ab0a (diff) | |
download | bcm5719-llvm-77b326f39d8b33615a57d41d868ae0bbcddd5bf4.tar.gz bcm5719-llvm-77b326f39d8b33615a57d41d868ae0bbcddd5bf4.zip |
Revert "Use an artifical namespace so that member vars do not hide local vars."
Summary:
This reverts commit 8af14b5f9af68c31ac80945e5b5d56f0a14b38e4.
Reverting as it breaks a few tests on Mac.
Reviewers: spyffe
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D16895
llvm-svn: 259823
Diffstat (limited to 'lldb/source/Expression/ExpressionSourceCode.cpp')
-rw-r--r-- | lldb/source/Expression/ExpressionSourceCode.cpp | 33 |
1 files changed, 2 insertions, 31 deletions
diff --git a/lldb/source/Expression/ExpressionSourceCode.cpp b/lldb/source/Expression/ExpressionSourceCode.cpp index 13fdaf8dfeb..93bac62c43c 100644 --- a/lldb/source/Expression/ExpressionSourceCode.cpp +++ b/lldb/source/Expression/ExpressionSourceCode.cpp @@ -16,9 +16,7 @@ #include "lldb/Symbol/DebugMacros.h" #include "lldb/Symbol/Block.h" #include "lldb/Symbol/TypeSystem.h" -#include "lldb/Symbol/VariableList.h" #include "lldb/Target/ExecutionContext.h" -#include "lldb/Target/Language.h" #include "lldb/Target/Platform.h" #include "lldb/Target/StackFrame.h" #include "lldb/Target/Target.h" @@ -177,21 +175,6 @@ AddMacros(const DebugMacros *dm, CompileUnit *comp_unit, AddMacroState &state, S } } -static void -AddLocalVariableDecls(const lldb::VariableListSP &var_list_sp, StreamString &stream) -{ - for (size_t i = 0; i < var_list_sp->GetSize(); i++) - { - lldb::VariableSP var_sp = var_list_sp->GetVariableAtIndex(i); - - ConstString var_name = var_sp->GetName(); - if (var_name == ConstString("this")) - continue; - - stream.Printf("using $__lldb_local_vars::%s;\n", var_name.AsCString()); - } -} - bool ExpressionSourceCode::GetText (std::string &text, lldb::LanguageType wrapping_language, bool const_object, bool static_method, ExecutionContext &exe_ctx) const { const char *target_specific_defines = "typedef signed char BOOL;\n"; @@ -256,7 +239,6 @@ bool ExpressionSourceCode::GetText (std::string &text, lldb::LanguageType wrappi } StreamString debug_macros_stream; - StreamString lldb_local_var_decls; if (StackFrame *frame = exe_ctx.GetFramePtr()) { const SymbolContext &sc = frame->GetSymbolContext( @@ -271,15 +253,8 @@ bool ExpressionSourceCode::GetText (std::string &text, lldb::LanguageType wrappi AddMacros(dm, sc.comp_unit, state, debug_macros_stream); } } - - ConstString object_name; - if (Language::LanguageIsCPlusPlus(frame->GetLanguage())) - { - lldb::VariableListSP var_list_sp = frame->GetInScopeVariableList(false); - AddLocalVariableDecls(var_list_sp, lldb_local_var_decls); - } } - + if (m_wrap) { switch (wrapping_language) @@ -309,23 +284,19 @@ bool ExpressionSourceCode::GetText (std::string &text, lldb::LanguageType wrappi wrap_stream.Printf("void \n" "%s(void *$__lldb_arg) \n" "{ \n" - " %s; \n" " %s; \n" "} \n", m_name.c_str(), - lldb_local_var_decls.GetData(), m_body.c_str()); break; case lldb::eLanguageTypeC_plus_plus: wrap_stream.Printf("void \n" "$__lldb_class::%s(void *$__lldb_arg) %s\n" "{ \n" - " %s; \n" " %s; \n" "} \n", m_name.c_str(), (const_object ? "const" : ""), - lldb_local_var_decls.GetData(), m_body.c_str()); break; case lldb::eLanguageTypeObjC: @@ -368,6 +339,6 @@ bool ExpressionSourceCode::GetText (std::string &text, lldb::LanguageType wrappi { text.append(m_body); } - + return true; } |