This is an archive of the discontinued LLVM Phabricator instance.

Fix TestRegisters on remote target.
ClosedPublic

Authored by chaoren on Feb 15 2015, 7:04 PM.

Reviewers

Commits

Summary

Using spawnSubprocess instead of forkSubprocess, so that the subprocess
spawns on the target and not the host.

chaoren retitled this revision from to Fix TestRegisters on remote target..

chaoren updated this object.

chaoren edited the test plan for this revision. (Show Details)

chaoren added a subscriber: Unknown Object (MLST).

ovyalov edited edge metadata.

This revision is now accepted and ready to land.Feb 15 2015, 8:15 PM

This revision was automatically updated to reflect the committed changes.

Looks good.

Path

Size

lldb/

trunk/

test/

functionalities/

7 lines

Show First 20 Lines • Show All 246 Lines • ▼ Show 20 Lines	def convenience_registers(self):
self.expect("register read rax 0x1234567887654321",		self.expect("register read rax 0x1234567887654321",
substrs = ['0x1234567887654321'])		substrs = ['0x1234567887654321'])

def convenience_registers_with_process_attach(self, test_16bit_regs):		def convenience_registers_with_process_attach(self, test_16bit_regs):
"""Test convenience registers after a 'process attach'."""		"""Test convenience registers after a 'process attach'."""
exe = os.path.join(os.getcwd(), "a.out")		exe = os.path.join(os.getcwd(), "a.out")

# Spawn a new process		# Spawn a new process
pid = 0		pid = self.spawnSubprocess(exe, ['wait_for_attach']).pid
if sys.platform.startswith('linux'):
pid = self.forkSubprocess(exe, ['wait_for_attach'])
else:
proc = self.spawnSubprocess(exe, ['wait_for_attach'])
pid = proc.pid
self.addTearDownHook(self.cleanupSubprocesses)		self.addTearDownHook(self.cleanupSubprocesses)

if self.TraceOn():		if self.TraceOn():
print "pid of spawned process: %d" % pid		print "pid of spawned process: %d" % pid

self.runCmd("process attach -p %d" % pid)		self.runCmd("process attach -p %d" % pid)

# Check that "register read eax" works.		# Check that "register read eax" works.
Show All 15 Lines