-
Notifications
You must be signed in to change notification settings - Fork 607
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
bpu: disable ittage when no indirect branch #2959
base: master
Are you sure you want to change the base?
Conversation
a45a742
to
8e003d0
Compare
[Generated by IPC robot]
master branch:
|
This PR may introduce timing violations. Wait for PT result. |
[Generated by IPC robot]
master branch:
|
RET is JALR too, so !isRet && isJalr. This condition is the same as ITTAGE updateValid, see ITTAGE.scala
On dhrystone, around 82% of BPU fire now does not use ITTAGE. However, ITTAGE is literally unused is dhrystone (see ittage_provided_at_pred below)
The other 20% ITTAGE usage is due to uftb miss when no branch is detected. The 20% can be further optimized after #2963 is ready. |
Quick and dirty approach. If the timing doesn't meet, will try ahead pipeline.