[Lldb-commits] [PATCH] D96463: [LLDB] Arm64/Linux test case for MTE and Pointer Authentication regset

David Spickett via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 22 07:42:20 PST 2021

DavidSpickett added a comment.

Some python nits otherwise the test looks good.

Comment at: lldb/test/API/commands/register/register/aarch64_dynamic_regset/TestArm64DynamicRegsets.py:16
+        self.assertTrue(reg_value.IsValid(),
+                        'Verify we have a register named "%s"' % (name))
+        self.assertEqual(reg_value.GetByteSize(), expected,
These messages are printed on assert failure. So this one would be "Expected a register named...".

Comment at: lldb/test/API/commands/register/register/aarch64_dynamic_regset/TestArm64DynamicRegsets.py:32
+                         for _ in range(z_reg_size)) + '}'
+            self.expect("register read " + 'z%i' %
+                        (i), substrs=[z_regs_value])
For this and the other instances of this pattern you an make it one string like:
self.expect("register read z%i" % i, ...

Comment at: lldb/test/API/commands/register/register/aarch64_dynamic_regset/TestArm64DynamicRegsets.py:45
+        self.expect("register read ffr", substrs=[p_regs_value])
Do we need an ffr read if we have a read/write below?

Comment at: lldb/test/API/commands/register/register/aarch64_dynamic_regset/TestArm64DynamicRegsets.py:94
+            if 'Scalable Vector Extension Registers' in registerSet.GetName():
+                if not self.isAArch64SVE():
+                    self.fail(
These if not could be self.assertFalse.



More information about the lldb-commits mailing list