aadsm added inline comments.

================
Comment at: 
lldb/test/API/functionalities/module_load_attach/TestModuleLoadAttach.py:32-33
+        # Continue so that dlopen is called.
+        breakpoint = self.target().BreakpointCreateBySourceRegex(
+            "// break after dlopen", lldb.SBFileSpec("main.c"))
+        self.assertNotEqual(breakpoint.GetNumResolvedLocations(), 0)
----------------
aadsm wrote:
> clayborg wrote:
> > Don't we need to break before the dlopen and make sure we don't have a 
> > libfeature.so in our module list, then run over the dlopen and verify we do 
> > see it afterwards? Wasn't this bug that we will see shared libraries 
> > correctly one time when we attach, but just not get any updates after this??
> that was a completely different bug and I have a different test for that 
> situation as well.
> Something that I could test though, is that before we got an update for an 
> unresolved breakpoint to make sure we did indeed transitioned from unresolved 
> -> resolved.
> 
> I'll add that.
Forget this, I shouldn't be answering comments in the morning.

> that was a completely different bug and I have a different test for that 
> situation as well.
> Something that I could test though, is that before we got an update for an 
> unresolved breakpoint to make sure we did indeed transitioned from unresolved 
> -> resolved.
> 
> I'll add that.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96637

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to