Skip to content

Commit

Permalink
Fix TestRegisters on remote target.
Browse files Browse the repository at this point in the history
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
  • Loading branch information
chaoren committed Feb 16, 2015
1 parent e7a9cdb commit a06c799
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions lldb/test/functionalities/register/TestRegisters.py
Expand Up @@ -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():
Expand Down

0 comments on commit a06c799

Please sign in to comment.