[Lldb-commits] [PATCH] D76188: [lldb/Target] Support more than 2 symbols in StackFrameRecognizer

Frederic Riss via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sun Mar 15 10:10:42 PDT 2020


friss added inline comments.


================
Comment at: lldb/test/API/commands/frame/recognizer/TestFrameRecognizer.py:162-167
+        lldbutil.run_break_set_by_symbol(self, "bar")
+        self.runCmd("c")
+
+        process = target.GetProcess()
+        thread = process.GetSelectedThread()
+        frame = thread.GetSelectedFrame()
----------------
Please rewrite using lldbutil.run_to_name_breakpoint, otherwise this LGTM


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76188/new/

https://reviews.llvm.org/D76188





More information about the lldb-commits mailing list