clayborg added a comment.
This change causes many test suite errors due to inline comment. Please fix ASAP
================
Comment at: lldb/packages/Python/lldbsuite/test/lldbutil.py:759
+
+ bkpt = test.target().FindBreakpointByID(bpno)
+ test.assertTrue(bkpt.IsValid(), "Breakpoint is not valid.")
----------------
This line throws and exception when I run it on my system. It seems we have a
function in lldbtest.py:
```
# utility methods that tests can use to access the current objects
def target(self):
if not self.dbg:
raise Exception('Invalid debugger instance')
return self.dbg.GetSelectedTarget()
```
And many tests actually set "self.target" to be a lldb.SBTarget with lines like:
```
self.target = self.dbg.CreateTarget(exe)
```
So this test fails for many objective C tests now with an exception
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111899/new/
https://reviews.llvm.org/D111899
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
