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

fix: prevent latch inference #135

Open
wants to merge 1 commit into
base: openpiton-dev
Choose a base branch
from

Conversation

tianrui-wei
Copy link
Contributor

This PR prevents latch inference by assigning a default value to msg_send_data_size_S4

Signed-off-by: Tianrui Wei <tianrui@tianruiwei.com>
@tianrui-wei
Copy link
Contributor Author

cc @Jbalkind @fei-g
The latch inference seems to come from a39066a. Would you have any insight on why the comments was inserted?

@fei-g
Copy link
Contributor

fei-g commented Oct 9, 2023

I don't remember exactly why did I comment that out.. Just checked the L15 decoder, and I found that it didn't look into the data_size field. I think in L15 pipeline it will use the data size stored in MSHR. So, we are okay to set the default data_size to 0. But If you have other modules that need the data_size info, the best way might be just uncomment those lines back (or set default data size to data_size_S4_f).

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