diff options
author | Tim Newsome <tim@sifive.com> | 2022-12-14 16:52:28 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-12-14 16:52:28 -0800 |
commit | 898c20bf0b74b2899d7491823cbbac84b5508751 (patch) | |
tree | cea2fb6d7176f4cfe14fc1d8f3bf2823a481fe55 /debug/gdbserver.py | |
parent | 45f4da6224ee254f235ff223f77e69dccf100c46 (diff) | |
download | riscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.zip riscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.tar.gz riscv-tests-898c20bf0b74b2899d7491823cbbac84b5508751.tar.bz2 |
debug: Remove unnecessary exit() functions. (#437)
Also make the semi-hosting test program return 10. That's more fragile
than returning 0, so makes for a better test.
Diffstat (limited to 'debug/gdbserver.py')
-rwxr-xr-x | debug/gdbserver.py | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/debug/gdbserver.py b/debug/gdbserver.py index a61722d..ac7b162 100755 --- a/debug/gdbserver.py +++ b/debug/gdbserver.py @@ -521,10 +521,7 @@ class DebugTest(GdbSingleHartTest): self.gdb.b("_exit") def exit(self, expected_result=0xc86455d4): - output = self.gdb.c() - assertIn("Breakpoint", output) - assertIn("_exit", output) - assertEqual(self.gdb.p("status"), expected_result) + super().exit(expected_result) class DebugCompareSections(DebugTest): def test(self): @@ -944,13 +941,6 @@ class Semihosting(GdbSingleHartTest): self.parkOtherHarts() self.gdb.b("_exit") - def exit(self, expected_result=0): - output = self.gdb.c() - assertIn("Breakpoint", output) - assertIn("_exit", output) - assertEqual(self.gdb.p("status"), expected_result) - return output - def test(self): with tempfile.NamedTemporaryFile(suffix=".data") as temp: self.gdb.b("main:begin") |