-
Notifications
You must be signed in to change notification settings - Fork 738
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
Verify VarHandle class initialization #4192
Conversation
7d27d8b
to
fff4da4
Compare
@JasonFengJ9 Do the tests pass on the RI as well? |
@DanHeidinga yes, I verified that |
Jenkins test sanity zlinux jdk11 |
@DanHeidinga the sanity build failure is a compilation error |
Jenkins test sanity zlinux,xlinux jdk11 |
Same failure again, it also occurred in another PR build https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK11-linux_390-64_cmprssptrs-OpenJ9/131/console, created #4206. |
@DanHeidinga could you start another PR build? |
@JasonFengJ9 you need to rebase first. |
When a static field hasn't been initialized implicitly and a VarHandle is created to access this field. The field has to be initialized before returning the value via the VarHandle. This test verifies that a VarHandle returns correct field value when there is no implicit static field initialization. Signed-off-by: Jason Feng <fengj@ca.ibm.com>
fff4da4
to
2b9c1a5
Compare
@pshipton yeah, that's right, done. |
Jenkins test sanity zlinux,xlinux jdk11 |
Verify
VarHandle
class initializationWhen a
static
field hasn't been initialized implicitly and aVarHandle
is created to access this field. The field has to be initialized before returning the value via theVarHandle
.This test verifies that a
VarHandle
returns correct field value when there is no implicit static field initialization.close #3205
depends #3928
Without dependent PR, this test fails with following stacktrace:
Manually verified that this test and the test from #3205 pass with #3928.
Reviewer: @DanHeidinga
FYI: @llxia
Signed-off-by: Jason Feng fengj@ca.ibm.com