Fuse reordered lw/sw instructions #240
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Originally, we only fused lw/sw insturciton with continuous memory address. However, some lw/sw instruction cannot be fused because of compiler reordering. For example: 108e8: 01312e23 sw s3,28(sp) 108ec: 02112623 sw ra,44(sp) 108f0: 02812423 sw s0,40(sp) 108f4: 02912223 sw s1,36(sp) 108f8: 03212023 sw s2,32(sp) 108fc: 01412c23 sw s4,24(sp) To fuse more lw/sw instructions taht reordered by compile, we need to relax the restriction. With this modification, we can fuse reordered lw/sw instructions.
jserv
changed the title
|
I observed a performance improvement on my host computer when running Dhrystone, but the benchmark results indicate a performance loss. |
vestata
pushed a commit
to vestata/rv32emu
that referenced
this pull request
Jan 24, 2025
Originally, we only fused LW/SW instructions with continuous memory addresses. However, some LW/SW instructions cannot be fused due to compiler instruction reordering. For example: 108e8: 01312e23 sw s3, 28(sp) 108ec: 02112623 sw ra, 44(sp) 108f0: 02812423 sw s0, 40(sp) 108f4: 02912223 sw s1, 36(sp) 108f8: 03212023 sw s2, 32(sp) 108fc: 01412c23 sw s4, 24(sp) To fuse more lw/sw instructions that are reordered by the compiler, we need to relax the restriction. With this modification, we can fuse reordered LW/SW instructions. Close sysprog21#218
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Originally, we only fused lw/sw insturciton with continuous memory address. However, some lw/sw instruction cannot be fused because of compiler reordering. For example:
To fuse more lw/sw instructions taht reordered by compile, we need to relax the restriction. With this modification, we can fuse reordered lw/sw instructions.
Close: #218