Skip to content

Conversation

@lachlanharrisdev
Copy link
Owner

@lachlanharrisdev lachlanharrisdev commented Sep 24, 2025

Description

This complete redesign switches the ISA used from LC-3 to RISC-V 32I. For a more detailed description of changes, please see individual commit messages

Checklist

Please ensure the following before requesting a merge:

  • I have read and agree to the repository's CONTRIBUTING.md and LICENSE.md.
  • All tests pass locally (make, make test).
  • Relevant documentation has been updated (README.md, docs/).
  • Code follows the repository's style and formatting guidelines.
  • I have added/updated relevant tests.

Related Issues

None provided.

Additional Notes

The docs/ directory has been deprecated in favour of GitHub wikis.


By submitting this pull request, you acknowledge the following:

  • Your contribution will be credited in the repository's contributor list and may be included in a major release.
  • Individual contributions may not be credited in associated research publications.
  • You confirm that you have the right to submit this work and that it does not violate any third-party rights or agreements.
  • All contributions are made under the terms of the repository's license.
  • If you violate any of the terms above or as outlined in CONTRIBUTING.md, your future contributions may not be accepted

- Updated utils.h to support 32-bit memory operations and removed obsolete functions.
- Deprecate hardware.c and instructions.c as part of the transition to a new architecture.
- Updated utililities in utils.c to use 32-bit rv32i compatible instructions
- Remove the bulk of test_utils.c temporarily
- Update compiler documentation to explain installation of toolchain. Note: still incomplete
@lachlanharrisdev lachlanharrisdev self-assigned this Sep 24, 2025
@lachlanharrisdev lachlanharrisdev added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 24, 2025
Copy link
Owner Author

@lachlanharrisdev lachlanharrisdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double checking nothing required for the future is deleted, looks clear to merge

@lachlanharrisdev lachlanharrisdev merged commit 5898e86 into main Sep 24, 2025
1 check passed
@lachlanharrisdev lachlanharrisdev deleted the riscv32i branch September 24, 2025 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants