Skip to content

Gate stores by predicate#311

Open
guosran wants to merge 1 commit into
tancheng:masterfrom
guosran:fix/memunit-predicate-store-rtl
Open

Gate stores by predicate#311
guosran wants to merge 1 commit into
tancheng:masterfrom
guosran:fix/memunit-predicate-store-rtl

Conversation

@guosran

@guosran guosran commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

Addresses #292 with a minimized store predicate fix. This PR only updates MemUnitRTL.py.

Changes

  • Gate store address/data valid signals with input predicates.
  • Prevent predicated-false stores from issuing memory writes.

Validation

Part of the master-based small conv passing set.

@guosran guosran marked this pull request as draft June 25, 2026 20:04
@guosran guosran marked this pull request as ready for review June 26, 2026 00:41
@guosran

guosran commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator Author

A store with predicate=0 should not write memory.

store addr = 5
store data = 123
predicate = 0

Before:
the store path could still issue a memory write.

After:
MemUnitRTL only sends the write when the store predicate is active.

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.

1 participant