Skip to content

fix s7/t7. fix pipe symbols on extended. Made pipe symbols show by default - #118

Merged
Spooky-Firefox merged 1 commit into
masterfrom
small_fixes
Oct 1, 2026
Merged

Spooky-Firefox merged 1 commit into
masterfrom
small_fixes

Conversation

@Spooky-Firefox

Copy link
Copy Markdown
Collaborator

This pull request makes improvements to the MIPS pipeline simulator by fixing register naming errors and enhancing support for dynamic symbols in instruction memory. The most significant changes are grouped below:

Register Naming Fixes

  • Corrected the order of register names in the REG_NAMES array in three files (mips_reg_file.rs, mips_mem_view_window.rs, and mips_reg_view_window.rs) to fix the placement of "t7" and "s7", ensuring registers are named according to the MIPS convention. [1] [2] [3]

Dynamic Symbol Support for Instruction Memory

  • Updated the construction of the InstrMem component in mips_extended_pipe_example.rs to use the with_dynamic_symbols method, enabling dynamic symbols for pipeline registers (PC_IM, PC_DE, PC_EX, PC_DM).
  • Modified the mips_pipe.json and mips_pipe_ex.json configuration files to set the dynamic flag to true for the pipeline registers (PC_DE, PC_EX, PC_DM, and PC_IM), ensuring these symbols are showed by default in the simulator. [1] [2] [3] [4]

@salon64 salon64 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtmdl!

@Spooky-Firefox
Spooky-Firefox merged commit 6efcefc into master Oct 1, 2026
4 checks passed
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.

2 participants