Skip to content

Commit 1776941

Browse files
committed
Revert debugging code
1 parent d007e05 commit 1776941

File tree

3 files changed

+8
-41
lines changed

3 files changed

+8
-41
lines changed

lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2543,7 +2543,8 @@ DWARFASTParserClang::ParseFunctionFromDWARF(CompileUnit &comp_unit,
25432543
// If the mangled name is not present in the DWARF, generate the
25442544
// demangled name using the decl context. We skip if the function is
25452545
// "main" as its name is never mangled.
2546-
func_name.SetValue(ConstructDemangledNameFromDWARF(die));
2546+
func_name.SetDemangledName(ConstructDemangledNameFromDWARF(die));
2547+
func_name.SetMangledName(ConstString(name));
25472548
} else
25482549
func_name.SetValue(ConstString(name));
25492550

lldb/source/Target/StackFrameRecognizer.cpp

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ StackFrameRecognizerManager::GetRecognizerForFrame(StackFrameSP frame) {
142142
Address start_addr = symbol->GetAddress();
143143
Address current_addr = frame->GetFrameCodeAddress();
144144

145-
for (const auto &entry : m_recognizers) {
145+
for (auto entry : m_recognizers) {
146146
if (entry.module)
147147
if (entry.module != module_name)
148148
continue;
@@ -151,8 +151,7 @@ StackFrameRecognizerManager::GetRecognizerForFrame(StackFrameSP frame) {
151151
if (!entry.module_regexp->Execute(module_name.GetStringRef()))
152152
continue;
153153

154-
ConstString function_name =
155-
symbol->GetMangled().GetName(entry.symbol_mangling);
154+
ConstString function_name = symctx.GetFunctionName(entry.symbol_mangling);
156155

157156
if (!entry.symbols.empty())
158157
if (!llvm::is_contained(entry.symbols, function_name))
@@ -168,14 +167,6 @@ StackFrameRecognizerManager::GetRecognizerForFrame(StackFrameSP frame) {
168167

169168
return entry.recognizer;
170169
}
171-
172-
LLDB_LOG(GetLog(LLDBLog::Symbols),
173-
"Did not recognize frame: module={0} symbol={1} mangled={2} "
174-
"demangled={3}",
175-
module_name, symbol->GetName(),
176-
symctx.GetFunctionName(Mangled::ePreferMangled),
177-
symctx.GetFunctionName(Mangled::ePreferDemangled));
178-
179170
return StackFrameRecognizerSP();
180171
}
181172

lldb/test/API/lang/swift/exception_breakpoint_recognizer/TestSwiftExceptionBreakpointRecognizer.py

Lines changed: 4 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -11,43 +11,18 @@ def test(self):
1111
target = lldbutil.run_to_breakpoint_make_target(self)
1212
bp = target.BreakpointCreateForException(lldb.eLanguageTypeSwift, False, True)
1313

14-
log_file = self.getBuildArtifact("symbol.log")
15-
self.runCmd(f"log enable lldb symbol -f {log_file}")
16-
1714
# First breakpoint in an untyped throws function.
1815
_, process, _, _ = lldbutil.run_to_breakpoint_do_run(self, target, bp)
1916
thread = process.selected_thread
2017
stop_desc = thread.GetStopDescription(128)
21-
# self.assertEqual(stop_desc, "Swift exception breakpoint")
18+
self.assertEqual(stop_desc, "Swift exception breakpoint")
2219
self.assertEqual(thread.frame[0].symbol.name, "swift_willThrow")
23-
# self.assertEqual(thread.selected_frame.idx, 1)
24-
if thread.selected_frame.idx != 1:
25-
r = lldb.SBCommandReturnObject()
26-
self.ci.HandleCommand("frame recognizer list", r)
27-
info = r.GetOutput()
28-
r = lldb.SBCommandReturnObject()
29-
self.ci.HandleCommand("frame recognizer info 0", r)
30-
info += r.GetOutput()
31-
with open(log_file) as f:
32-
log_lines = f.readlines()
33-
info += "".join(line for line in log_lines if "recognize" in line)
34-
self.assertIn("qowieoiuqweoiq", info)
20+
self.assertEqual(thread.selected_frame.idx, 1)
3521

3622
# Second breakpoint in an typed throws function.
3723
process.Continue()
3824
thread = process.selected_thread
3925
stop_desc = thread.GetStopDescription(128)
40-
# self.assertEqual(stop_desc, "Swift exception breakpoint")
26+
self.assertEqual(stop_desc, "Swift exception breakpoint")
4127
self.assertEqual(thread.frame[0].symbol.name, "swift_willThrowTypedImpl")
42-
# self.assertEqual(thread.selected_frame.idx, 2)
43-
if thread.selected_frame.idx != 2:
44-
r = lldb.SBCommandReturnObject()
45-
self.ci.HandleCommand("frame recognizer list", r)
46-
info = r.GetOutput()
47-
r = lldb.SBCommandReturnObject()
48-
self.ci.HandleCommand("frame recognizer info 0", r)
49-
info += r.GetOutput()
50-
with open(log_file) as f:
51-
log_lines = f.readlines()
52-
info += "".join(line for line in log_lines if "recognize" in line)
53-
self.assertIn("qowieoiuqweoiq", info)
28+
self.assertEqual(thread.selected_frame.idx, 2)

0 commit comments

Comments
 (0)