openj9
openj9 copied to clipboard
WIP: Load default value instance if the class is initialized
ILGen:
- aconst_init: Load the default value instance if the class is initialized. This enhancement can be disabled
by setting env variable
TR_DisableLoadStaticDefaultValueInstance.
SymRef:
- Create symbol reference for default value instance of value class.
AOT (X86):
- Add new relocation record
TR_RelocationRecordStaticDefaultValueInstanceto materialize the default value instance slot address on AOT load.
Depends on
- [ ] https://github.com/eclipse/omr/pull/6641
Signed-off-by: Annabelle Huo [email protected]
This PR has dependency on https://github.com/eclipse/omr/pull/6641 and should be reviewed together with it
@dsouzai @hzongaro May I ask you to review this change along with https://github.com/eclipse/omr/pull/6641? Thank you!
@hzongaro @dsouzai All comments are addressed. Ready for another review. Thanks!
Removed WIP. The OMR dependency https://github.com/eclipse/omr/pull/6641 has been merged to https://github.com/eclipse-openj9/openj9-omr. This PR is ready for another review and PR build test.
AOT changes look good to me. @hzongaro do you mind being committer for this one?
Jenkins test sanity all jdk17
@hzongaro For this PR, JDKNext standard and JDKNext valhalla should be tested as well. You probably already plan to do so. Just a reminder
Jenkins test sanity,extended xlinuxval jdknext
xlinuxval build failure appears to be unrelated to this pull request.
Jenkins test sanity+aot xlinux jdknext
Jenkins test sanity,extended xlinuxval jdknext