clayborg added inline comments.

================
Comment at: 
lldb/test/API/functionalities/module_load_attach/TestModuleLoadAttach.py:28
+    def test_x(self):
+        process = self.build_launch_and_attach()
+        thread = process.GetSelectedThread()
----------------
Is this racy? What happens on a really slow system? Can we fail to attach? If 
we do attach, are we guaranteed to be at a place where we can set 
"flip_to_1_to_continue = 1"? The nice thing is it is a global variable that we 
should be able to set no matter where we stop. 


================
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)
----------------
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??


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