[Lldb-commits] [PATCH] D146965: [lldb] Add support for MSP430 in LLDB.

David Spickett via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 13 01:57:23 PDT 2023

DavidSpickett added a comment.

Thanks for the test changes, looks good.

There is some documentation that lists what targets we support, MSP430 should be added there. `lldb/docs/index.rst` is one of those.

Otherwise, can anyone else think of major debug features that should be (at least) smoke tested? (of course you could do this in follow up patches if it needs more work)

Comment at: lldb/test/API/functionalities/gdb_remote_client/TestMSP430MSPDebug.py:103
+            self.assertEqual(reg.GetValueAsUnsigned(),
+                             reg_val_dict[reg.GetName()])
Write some registers too? Again, not likely to find anything, just ensures we have the right offsets and don't take the wrong 16 bits out of the value.



More information about the lldb-commits mailing list