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

[David Spickett via Phabricator via lldb-commits]

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.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96463/new/

https://reviews.llvm.org/D96463