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