Skip to content

Fixing LoopBuffer bug#421

Merged
ABenC377 merged 7 commits intodevfrom
LoopBuffer
Aug 2, 2024
Merged

Fixing LoopBuffer bug#421
ABenC377 merged 7 commits intodevfrom
LoopBuffer

Conversation

@ABenC377
Copy link
Copy Markdown
Contributor

Updates how a LoopBuffer is begun; and how branch mispredicts are determined by the execute units.

This was motivated by the incorrect misprediction calculation in the execute. However, doing so resulted in a bug. The changes to when the LoopBuffer is used by the Fetch Unit fixes this bug.

@ABenC377 ABenC377 self-assigned this Jul 25, 2024
@ABenC377 ABenC377 added the bug Something isn't working label Jul 25, 2024
FinnWilkinson
FinnWilkinson previously approved these changes Jul 25, 2024
FinnWilkinson
FinnWilkinson previously approved these changes Jul 25, 2024
FinnWilkinson
FinnWilkinson previously approved these changes Jul 25, 2024
dANW34V3R
dANW34V3R previously approved these changes Jul 31, 2024
Copy link
Copy Markdown
Contributor

@dANW34V3R dANW34V3R left a comment

Choose a reason for hiding this comment

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

Great find

Comment thread src/lib/pipeline/FetchUnit.cc Outdated
Comment thread src/lib/pipeline/FetchUnit.cc Outdated
@ABenC377 ABenC377 merged commit 1c394e7 into dev Aug 2, 2024
@ABenC377 ABenC377 deleted the LoopBuffer branch August 2, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants