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

Add jdk_lang_VarHandleTest_j9 test #3100

Merged
merged 1 commit into from
Oct 27, 2021
Merged

Conversation

LongyuZhang
Copy link
Contributor

Signed-off-by: Longyu Zhang longyu.zhang@ibm.com

@LongyuZhang
Copy link
Contributor Author

Test Links:
plinux: 19034 zlinux: 19048 aix: 19050 mac: 19051 win: 19052

-w $(Q)$(REPORTDIR)$(D)work$(Q) \
-r $(Q)$(TEST_RESROOT)$(D)report$(Q) \
-jdk:$(Q)$(TEST_JDK_HOME)$(Q) \
$(Q)$(OPENJDK_DIR)$(D)test$(D)jdk$(D)java$(D)lang$(D)invoke$(D)VarHandles$(Q); \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$(D) is not necessary. We do not need to use it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to use /, thanks

<group>openjdk</group>
</groups>
<impls>
<impl>openj9</impl>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add ibm

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added ibm too.

@@ -577,6 +577,33 @@
<impl>ibm</impl>
</impls>
</test>
<test>
<testCaseName>jdk_lang_VarHandleTest_j9</testCaseName>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • jdk_lang_VarHandleTest_j9 needs to be excluded in jdk_lang
  • jit opt is a temporary solution. Please add a comment about it

Copy link
Contributor Author

@LongyuZhang LongyuZhang Oct 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment about temporary solution with jit opt has been added.
Excluding VarHandleTests are in another PR: #3101.
New test links: win: 19057 plinux: 19058

- Add jdk_lang_VarHandleTest_j9 test with specific jit option

Signed-off-by: Longyu Zhang <longyu.zhang@ibm.com>
Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @LongyuZhang

@llxia llxia requested a review from renfeiw October 27, 2021 16:02
Copy link
Contributor

@renfeiw renfeiw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Successfully merging this pull request may close these issues.

3 participants