Skip to content

Commit

Permalink
Support multiple jdk custom targets (#4136)
Browse files Browse the repository at this point in the history
* Support multiple jdk custom target

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>

* Remove unnecessary if condition

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
  • Loading branch information
sophia-guo committed Nov 10, 2022
1 parent 34dc112 commit 9e3b5f6
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 5 additions & 3 deletions openjdk/openjdk.mk
Original file line number Diff line number Diff line change
Expand Up @@ -119,9 +119,6 @@ else
OPENJDK_DIR := $(TEST_ROOT)$(D)openjdk$(D)openjdk-jdk
endif

JDK_CUSTOM_TARGET ?= java/math/BigInteger/BigIntegerTest.java
HOTSPOT_CUSTOM_TARGET ?= gc/stress/gclocker/TestExcessGCLockerCollections.java
LANGTOOLS_CUSTOM_TARGET ?= tools/javac/4241573/T4241573.java
ifneq (,$(findstring $(JDK_VERSION),8-9))
JTREG_JDK_TEST_DIR := $(OPENJDK_DIR)$(D)jdk$(D)test
JTREG_HOTSPOT_TEST_DIR := $(OPENJDK_DIR)$(D)hotspot$(D)test
Expand All @@ -132,6 +129,11 @@ else
JTREG_LANGTOOLS_TEST_DIR := $(OPENJDK_DIR)$(D)test$(D)langtools
endif

JDK_CUSTOM_TARGET ?= java/math/BigInteger/BigIntegerTest.java
HOTSPOT_CUSTOM_TARGET ?= gc/stress/gclocker/TestExcessGCLockerCollections.java
LANGTOOLS_CUSTOM_TARGET ?= tools/javac/4241573/T4241573.java
FULLPATH_JDK_CUSTOM_TARGET = $(foreach target,$(JDK_CUSTOM_TARGET),$(JTREG_JDK_TEST_DIR)$(D)$(target))

JDK_NATIVE_OPTIONS :=
JVM_NATIVE_OPTIONS :=
CUSTOM_NATIVE_OPTIONS :=
Expand Down
2 changes: 1 addition & 1 deletion openjdk/playlist.xml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
-w $(Q)$(REPORTDIR)$(D)work$(Q) \
-r $(Q)$(REPORTDIR)$(D)report$(Q) \
-jdk:$(Q)$(TEST_JDK_HOME)$(Q) \
$(Q)$(JTREG_JDK_TEST_DIR)$(D)$(JDK_CUSTOM_TARGET)$(Q); \
$(FULLPATH_JDK_CUSTOM_TARGET); \
$(TEST_STATUS)</command>
<levels>
<level>extended</level>
Expand Down

0 comments on commit 9e3b5f6

Please sign in to comment.