[Lldb-commits] [PATCH] D81200: [vscode] set default values for terminateDebuggee for the disconnect request

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 5 02:09:24 PDT 2020

labath added inline comments.

Comment at: lldb/test/API/tools/lldb-vscode/disconnect/TestVSCode_disconnect.py:55
+        self.process = subprocess.Popen([program, "--attach"],
+                                        universal_newlines=True,
Writing an attach test is tricky because of the yama restrictions on linux. The way that we handle that right now is to have the inferior call `lldb_enable_attach()` and then create a token file. The test then waits for the token file `lldbutil.wait_for_file_on_target` and only then attaches. This ensures the attack takes place only when yama is disabled.

Comment at: lldb/test/API/tools/lldb-vscode/disconnect/TestVSCode_disconnect.py:68
+        self.disconnect_and_check_no_output()
I'm confused as to why we're not expecting any output here. I was under the impression that we want to let the inferior continue in the "attach" case.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list