Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tool to test DSP instructions #70

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

JayFoxRox
Copy link
Owner

@JayFoxRox JayFoxRox commented Dec 26, 2018

This should make it easier to test DSP instruction behaviour.

Can be ran like this with debug bios:

A56=/home/user/a56/a56 XBOX=localhost:8731 ./dsp_instruction_test.py dsp_instruction_tests/*-test.json 

(Requires a56; example was with -redir tcp:8731::731 in XQEMU CLI)

For test syntax, check the examples. Further changes to input / output syntax are likely.


Most critical missing features / known issues:

  • Memory can not be set as input / output, no fatal error is generated
  • Some registers are blacklisted / ignored, no fatal error is generated
  • DSP assembler errors are silently ignored, no fatal error is generated
  • JSON and assembly parser issues are not reported properly
  • Output folder is same as input folder, so it's hard to generate 2 outputs for different machines
  • No tool to compare JSON output files
  • Code is ugly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant