Skip to content

[LSQ] Minor fixes and simplifications#993

Merged
MaxWipfli merged 4 commits into
mainfrom
mwipfli/lsq-fixes
Jul 6, 2026
Merged

[LSQ] Minor fixes and simplifications#993
MaxWipfli merged 4 commits into
mainfrom
mwipfli/lsq-fixes

Conversation

@MaxWipfli

Copy link
Copy Markdown
Collaborator

This includes a number of small fixes and improvements to the LSQ generator that have been cherry-picked from my semester project code.

  • Clean up computation of empty{Ld,St}AddrW in the configuration
  • Add assertions to ensure load/store queues are large enough so that all groups can be allocated.
  • Add missing pipeline register ldq_head_oh_pcomp
  • Simplify the IntToBits helper function and add clear error messages (e.g., for negative inputs)

@MaxWipfli MaxWipfli requested a review from murphe67 July 1, 2026 10:48
@MaxWipfli

Copy link
Copy Markdown
Collaborator Author

cc @JasmijnB regarding #914

@murphe67 murphe67 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.

Thanks for pulling these out of the rest of your stuff Max 😁

@MaxWipfli MaxWipfli merged commit 7a19e08 into main Jul 6, 2026
8 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