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.

Sign up for GitHub

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

Jump to bottom

[TASK] Remove Rename hardware module #1370

Closed
fatimasaleem opened this issue Sep 7, 2023 · 1 comment
Closed

[TASK] Remove Rename hardware module #1370

fatimasaleem opened this issue Sep 7, 2023 · 1 comment
Assignees
@fatimasaleem
Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Task Project related task

Comments

@fatimasaleem
Copy link
Contributor

Description

Rename creates a 5% performance increase. Previously rename was done by hardware, by enabling the rename module. From the time we use GCC13, the rename is done by compilation. This module is currently inefficient and can be removed.

Changes

Update the issue logic to pass the decoder signals directly to issue_read_operand and scoreboard blocks
Remove RenameEn parameter from the configurations

Current Status

PR created

Risks

None

Prerequisites

None

KPI

None

Description of Done

PR of design changes with clean CI
@fatimasaleem fatimasaleem converted this from a draft issue Sep 7, 2023
@fatimasaleem fatimasaleem self-assigned this Sep 8, 2023
@fatimasaleem fatimasaleem added Component:RTL For issues in the RTL (e.g. for files in the rtl directory) Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Task Project related task labels Sep 8, 2023
@JeanRochCoulon JeanRochCoulon added CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration labels Sep 12, 2023
@fatimasaleem fatimasaleem moved this from Done - Ready for Review to Accepted by Reviewer in CVA6 Project Task Board Sep 15, 2023
@fatimasaleem
Copy link
Contributor Author

Completed in #1359

@github-project-automation github-project-automation bot moved this from Accepted by Reviewer to Done - Ready for Review in CVA6 Project Task Board Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fatimasaleem fatimasaleem

Labels
Component:RTL For issues in the RTL (e.g. for files in the rtl directory) CV32A60AX Part: Application configuration (former "step1") CV32A65X Part: Embedded configuration Status:Resolved Issue has been resolved, but closure is pending on git merge and/or issuer confirmation Type:Task Project related task
Projects
CVA6 Project Task Board
Status: Done - Ready for Review
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeanRochCoulon @fatimasaleem