shafik updated this revision to Diff 219604.
shafik marked an inline comment as done.
shafik added a comment.
- Refactored test to use a function to do repetitive task
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67111/new/
https://reviews.llvm.org/D67111
Files:
packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/TestLibCxxFunction.py
packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/main.cpp
source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.cpp
source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.h
Index: source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.h
===================================================================
--- source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.h
+++ source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.h
@@ -10,6 +10,9 @@
#define liblldb_CPPLanguageRuntime_h_
#include <vector>
+
+#include "llvm/ADT/StringMap.h"
+
#include "lldb/Core/PluginInterface.h"
#include "lldb/Target/LanguageRuntime.h"
#include "lldb/lldb-private.h"
@@ -82,6 +85,11 @@
CPPLanguageRuntime(Process *process);
private:
+ using OperatorStringToCallableInfoMap =
+ llvm::StringMap<CPPLanguageRuntime::LibCppStdFunctionCallableInfo>;
+
+ OperatorStringToCallableInfoMap CallableLookupCache;
+
DISALLOW_COPY_AND_ASSIGN(CPPLanguageRuntime);
};
Index: source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.cpp
===================================================================
--- source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.cpp
+++ source/Plugins/LanguageRuntime/CPlusPlus/CPPLanguageRuntime.cpp
@@ -193,13 +193,30 @@
symbol = sc.symbol;
}
+ // Case 4 or 5
+ // We eliminate these cases early because they don't need the potentially
+ // expensive lookup through the symbol table.
+ if (symbol && !symbol->GetName().GetStringRef().startswith("vtable for") &&
+ !first_template_parameter.contains("$_") &&
+ !first_template_parameter.contains("'lambda'") &&
+ !symbol->GetName().GetStringRef().contains("__invoke")) {
+ optional_info.callable_case =
+ LibCppStdFunctionCallableCase::FreeOrMemberFunction;
+ optional_info.callable_address = function_address_resolved;
+ optional_info.callable_symbol = *symbol;
+
+ return optional_info;
+ }
+
auto get_name = [&first_template_parameter, &symbol]() {
// Given case 1:
//
// main::$_0
+ // Bar::add_num2(int)::'lambda'(int)
//
// we want to append ::operator()()
- if (first_template_parameter.contains("$_"))
+ if (first_template_parameter.contains("$_") ||
+ first_template_parameter.contains("'lambda'"))
return llvm::Regex::escape(first_template_parameter.str()) +
R"(::operator\(\)\(.*\))";
@@ -228,6 +245,9 @@
std::string func_to_match = get_name();
+ if (CallableLookupCache.count(func_to_match))
+ return CallableLookupCache[func_to_match];
+
SymbolContextList scl;
target.GetImages().FindSymbolsMatchingRegExAndType(
@@ -249,6 +269,7 @@
addr.CalculateSymbolContextLineEntry(line_entry);
if (first_template_parameter.contains("$_") ||
+ first_template_parameter.contains("'lambda'") ||
(symbol != nullptr &&
symbol->GetName().GetStringRef().contains("__invoke"))) {
// Case 1 and 2
@@ -262,19 +283,10 @@
optional_info.callable_symbol = *symbol;
optional_info.callable_line_entry = line_entry;
optional_info.callable_address = addr;
- return optional_info;
}
}
- // Case 4 or 5
- if (symbol && !symbol->GetName().GetStringRef().startswith("vtable for")) {
- optional_info.callable_case =
- LibCppStdFunctionCallableCase::FreeOrMemberFunction;
- optional_info.callable_address = function_address_resolved;
- optional_info.callable_symbol = *symbol;
-
- return optional_info;
- }
+ CallableLookupCache[func_to_match] = optional_info;
return optional_info;
}
Index: source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
===================================================================
--- source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
+++ source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
@@ -568,6 +568,11 @@
"weak_ptr synthetic children",
ConstString("^(std::__[[:alnum:]]+::)weak_ptr<.+>(( )?&)?$"),
stl_synth_flags, true);
+ AddCXXSummary(cpp_category_sp,
+ lldb_private::formatters::LibcxxFunctionSummaryProvider,
+ "libc++ std::function summary provider",
+ ConstString("^std::__[[:alnum:]]+::function<.+>$"),
+ stl_summary_flags, true);
stl_summary_flags.SetDontShowChildren(false);
stl_summary_flags.SetSkipPointers(false);
Index: packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/main.cpp
===================================================================
--- packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/main.cpp
+++ packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/main.cpp
@@ -17,8 +17,25 @@
return 66 ;
}
int add_num(int i) const { return i + 3 ; }
+ int add_num2(int i) {
+ std::function<int (int)> add_num2_f = [](int x) {
+ return x+1;
+ };
+
+ return add_num2_f(i); // Set break point at this line.
+ }
} ;
+int foo2() {
+ auto f = [](int x) {
+ return x+1;
+ };
+
+ std::function<int (int)> foo2_f = f;
+
+ return foo2_f(10); // Set break point at this line.
+}
+
int main (int argc, char *argv[])
{
int acc = 42;
@@ -35,5 +52,8 @@
std::function<int ()> f4( bar1 ) ;
std::function<int (const Bar&, int)> f5 = &Bar::add_num;
+ int foo2_result = foo2();
+ int bar_add_num2_result = bar1.add_num2(10);
+
return f1(acc,acc) + f2(acc) + f3(acc+1,acc+2) + f4() + f5(bar1, 10); // Set break point at this line.
}
Index: packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/TestLibCxxFunction.py
===================================================================
--- packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/TestLibCxxFunction.py
+++ packages/Python/lldbsuite/test/functionalities/data-formatter/data-formatter-stl/libcxx/function/TestLibCxxFunction.py
@@ -15,11 +15,21 @@
mydir = TestBase.compute_mydir(__file__)
- def get_variable(self, name):
- var = self.frame().FindVariable(name)
- var.SetPreferDynamicValue(lldb.eDynamicCanRunTarget)
- var.SetPreferSyntheticValue(True)
- return var
+ # Run frame var for a variable twice. Verify we do not hit the cache
+ # the first time but do the second time.
+ def run_frame_var_check_cache_use(self, variable, result_to_match):
+ self.runCmd("log timers reset")
+ self.expect("frame variable " + variable,
+ substrs=[variable + " = " + result_to_match])
+ self.expect("log timers dump",
+ substrs=["lldb_private::Module::FindSymbolsMatchingRegExAndType"])
+
+ self.runCmd("log timers reset")
+ self.expect("frame variable " + variable,
+ substrs=[variable + " = " + result_to_match])
+ self.expect("log timers dump",
+ patterns=["(?!lldb_private::Module::FindSymbolsMatchingRegExAndType).*"])
+
# Temporarily skipping for everywhere b/c we are disabling the std::function formatter
# due to performance issues but plan on turning it back on once they are addressed.
@@ -41,17 +51,22 @@
substrs=['stopped',
'stop reason = breakpoint'])
- self.expect("frame variable f1",
- substrs=['f1 = Function = foo(int, int)'])
+ self.run_frame_var_check_cache_use("foo2_f", "Lambda in File main.cpp at Line 30")
+
+ lldbutil.continue_to_breakpoint(self.process(), bkpt)
- self.expect("frame variable f2",
- substrs=['f2 = Lambda in File main.cpp at Line 26'])
+ self.run_frame_var_check_cache_use("add_num2_f", "Lambda in File main.cpp at Line 21")
- self.expect("frame variable f3",
- substrs=['f3 = Lambda in File main.cpp at Line 30'])
+ lldbutil.continue_to_breakpoint(self.process(), bkpt)
- self.expect("frame variable f4",
- substrs=['f4 = Function in File main.cpp at Line 16'])
+ self.run_frame_var_check_cache_use("f2", "Lambda in File main.cpp at Line 43")
+ self.run_frame_var_check_cache_use("f3", "Lambda in File main.cpp at Line 47")
+ self.run_frame_var_check_cache_use("f4", "Function in File main.cpp at Line 16")
+
+ # These cases won't hit the cache at all but also don't require
+ # an expensive lookup.
+ self.expect("frame variable f1",
+ substrs=['f1 = Function = foo(int, int)'])
self.expect("frame variable f5",
substrs=['f5 = Function = Bar::add_num(int) const'])
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
