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
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
