[Lldb-commits] [lldb] r229357 - Fix TestRegisters on remote target.

Chaoren Lin chaorenl at google.com
Sun Feb 15 22:17:51 PST 2015


Author: chaoren
Date: Mon Feb 16 00:17:51 2015
New Revision: 229357

URL: http://llvm.org/viewvc/llvm-project?rev=229357&view=rev
Log:
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

Modified:
    lldb/trunk/test/functionalities/register/TestRegisters.py

Modified: lldb/trunk/test/functionalities/register/TestRegisters.py
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/functionalities/register/TestRegisters.py?rev=229357&r1=229356&r2=229357&view=diff
==============================================================================
--- lldb/trunk/test/functionalities/register/TestRegisters.py (original)
+++ lldb/trunk/test/functionalities/register/TestRegisters.py Mon Feb 16 00:17:51 2015
@@ -252,12 +252,7 @@ class RegisterCommandsTestCase(TestBase)
         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():





More information about the lldb-commits mailing list