-
Notifications
You must be signed in to change notification settings - Fork 29
[AIE2P] Map same banks for input and output ops of G_BITCAST #422
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
Conversation
|
This looks really good. Can you rebase end re-run QoR in face of new Reg Bank Select changes? |
|
Looking at the regressions, |
Now rebasing to the latest regbank changes, we have more improvements to stack size in more kernels. |
d65574a to
e286f8b
Compare
e286f8b to
7b1ec4b
Compare
andcarminati
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
[AutoBump] Merge with fixes of e6eb94d (Sep 20) (8)
Mapping G_BITCAST ops with same bank is more beneficial.
qor