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 disabled loads of constants in AOT #9106

Open
dsouzai opened this issue Apr 3, 2020 · 2 comments
Open

Enable disabled loads of constants in AOT #9106

dsouzai opened this issue Apr 3, 2020 · 2 comments

Comments

@dsouzai
Copy link
Contributor

dsouzai commented Apr 3, 2020

#9071 prevents AOT compilation of methods that have Constant Dynamic, Method Handle Constant, or Method Type Constant loads. This issue tracks enabling this functionality.

The idea would be (to quote @andrewcraik ) "[to disable] the code that would fold these so that we just leave them as loads. They would need a relocation to load them from the constant pool, but assuming we did that then it should work, just the scope of optimization will be limited because we cannot treat them as constant."

@mstoodle
Copy link
Contributor

mstoodle commented Apr 7, 2020

somewhat misleading title :)

@dsouzai
Copy link
Contributor Author

dsouzai commented Apr 8, 2020

Heh sorry, the character limit on Issue/PR titles makes for some...creative...choice of words :)

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