From a06c7997290d5b083da72132769099c02edd71e0 Mon Sep 17 00:00:00 2001 From: Chaoren Lin Date: Mon, 16 Feb 2015 06:17:51 +0000 Subject: [PATCH] Fix TestRegisters on remote target. Summary: Using spawnSubprocess instead of forkSubprocess, so that the subprocess spawns on the target and not the host. Requires http://reviews.llvm.org/D7660 for cleanup. Test Plan: TestRegisters.py passing. Reviewers: ovyalov, vharron, clayborg Subscribers: lldb-commits Differential Revision: http://reviews.llvm.org/D7661 llvm-svn: 229357 --- lldb/test/functionalities/register/TestRegisters.py | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/lldb/test/functionalities/register/TestRegisters.py b/lldb/test/functionalities/register/TestRegisters.py index 822aa0b5a2bba..0580c201f7808 100644 --- a/lldb/test/functionalities/register/TestRegisters.py +++ b/lldb/test/functionalities/register/TestRegisters.py @@ -252,12 +252,7 @@ def convenience_registers_with_process_attach(self, test_16bit_regs): exe = os.path.join(os.getcwd(), "a.out") # Spawn a new process - pid = 0 - if sys.platform.startswith('linux'): - pid = self.forkSubprocess(exe, ['wait_for_attach']) - else: - proc = self.spawnSubprocess(exe, ['wait_for_attach']) - pid = proc.pid + pid = self.spawnSubprocess(exe, ['wait_for_attach']).pid self.addTearDownHook(self.cleanupSubprocesses) if self.TraceOn():