Skip to content

Commit ff54a64

Browse files
author
Doug Simon
committed
8360049: CodeInvalidationReasonTest.java fails with ZGC on AArch64
Reviewed-by: aph, shade
1 parent c11f36e commit ff54a64

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

test/hotspot/jtreg/ProblemList-zgc.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,5 +125,3 @@ compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/NativeCallTest.java
125125
compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/SimpleCodeInstallationTest.java 8343233 generic-aarch64
126126
compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/SimpleDebugInfoTest.java 8343233 generic-aarch64
127127
compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/VirtualObjectDebugInfoTest.java 8343233 generic-aarch64
128-
129-
compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/CodeInvalidationReasonTest.java 8360049 generic-aarch64

test/hotspot/jtreg/compiler/jvmci/jdk.vm.ci.code.test/src/jdk/vm/ci/code/test/TestHotSpotVMConfig.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,10 +67,11 @@ private Boolean initNmethodEntryBarrierConcurrentPatch(Architecture arch) {
6767
// There currently only 2 variants in use that differ only by the presence of a
6868
// dmb instruction
6969
int stw = getConstant("NMethodPatchingType::stw_instruction_and_data_patch", Integer.class);
70-
int conc = getConstant("NMethodPatchingType::conc_data_patch", Integer.class);
70+
int conc1 = getConstant("NMethodPatchingType::conc_data_patch", Integer.class);
71+
int conc2 = getConstant("NMethodPatchingType::conc_instruction_and_data_patch", Integer.class);
7172
if (patchingType == stw) {
7273
patchConcurrent = false;
73-
} else if (patchingType == conc) {
74+
} else if (patchingType == conc1 || patchingType == conc2) {
7475
patchConcurrent = true;
7576
} else {
7677
throw new IllegalArgumentException("unsupported barrier sequence " + patchingType);

0 commit comments

Comments
 (0)