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.

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

Enable RegDepCopyRemoval on Z #12196

Open
fjeremic opened this issue Mar 11, 2021 · 2 comments
Open

Enable RegDepCopyRemoval on Z #12196

fjeremic opened this issue Mar 11, 2021 · 2 comments

Comments

@fjeremic
Copy link
Contributor

See #12188. We should investigate whether this is beneficial on Z and enable it if deemed so.

@r30shah
Copy link
Contributor

r30shah commented Mar 11, 2021

I remember having discussions about this with Andrew last year when working on to fix some issues with this Opt and PostGRA Block splitter. At that time while evaluating, on x86 this optimization gave me 0.3-0.4% improvement on Liberty-DayTrader with Java8.
One of the potential reason, we are getting something out of this optimization on x86 more is because the less number of Global Registers available on x86 causing more register spilling which could not be the case for Z where we have more Global Registers available.

@fjeremic
Copy link
Contributor Author

@0xdaryl FYI on above. Not sure if AArch64 has similar issues. It may not be worth the investment if AArch64 falls in the same bucket as Z w.r.t. global register usage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants