From cd05406b105334b01f0eae141e7e05d6fb4a2ecb Mon Sep 17 00:00:00 2001 From: Davide Italiano Date: Mon, 20 Jul 2020 14:11:58 -0700 Subject: [PATCH] [testsuite] Adapt lldb-server base test helper to run on arm64 --- .../lldbsuite/test/tools/lldb-server/gdbremote_testcase.py | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py b/lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py index d702d8e..71e6fdd 100644 --- a/lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py +++ b/lldb/packages/Python/lldbsuite/test/tools/lldb-server/gdbremote_testcase.py @@ -1537,7 +1537,7 @@ class GdbRemoteTestCaseBase(TestBase): g_c2_address = int(context.get("g_c2_address"), 16) # Set a breakpoint at the given address. - if self.getArchitecture() == "arm": + if self.getArchitecture().startswith("arm"): # TODO: Handle case when setting breakpoint in thumb code BREAKPOINT_KIND = 4 else: @@ -1601,8 +1601,18 @@ class GdbRemoteTestCaseBase(TestBase): # variable value if re.match("s390x", arch): expected_step_count = 2 + # ARM64 requires "4" instructions: 2 to compute the address (adrp, add), + # one to materialize the constant (mov) and the store + if re.match("arm64", arch): + expected_step_count = 4 + self.assertEqual(step_count, expected_step_count) + # ARM64: Once addresses and constants are materialized, only one + # instruction is needed. + if re.match("arm64", arch): + expected_step_count = 1 + # Verify we hit the next state. args["expected_g_c1"] = "0" args["expected_g_c2"] = "0" -- 2.7.4