Skip to content
Permalink
Browse files
8257674: [lworld] C1's substitutability check does not respect UseCom…
…pressedClassPointers
  • Loading branch information
TobiHartmann committed Dec 3, 2020
1 parent 8408e30 commit 330f2f2924e3ecf56df442d4b079f20db728d4a7
Showing with 1 addition and 3 deletions.
  1. +1 −1 src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp
  2. +0 −2 test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestLWorld.java
@@ -2078,7 +2078,7 @@ void LIR_Assembler::emit_opSubstitutabilityCheck(LIR_OpSubstitutabilityCheck* op
Register left_klass_op = op->left_klass_op()->as_register();
Register right_klass_op = op->right_klass_op()->as_register();

if (UseCompressedOops) {
if (UseCompressedClassPointers) {
__ movl(left_klass_op, Address(left, oopDesc::klass_offset_in_bytes()));
__ movl(right_klass_op, Address(right, oopDesc::klass_offset_in_bytes()));
__ cmpl(left_klass_op, right_klass_op);
@@ -2269,8 +2269,6 @@ public void test84_verifier(boolean warmup) {
MyValue2[] src = new MyValue2[100];
Arrays.fill(src, testValue2);
MyValue2[] dst = new MyValue2[100];
Method m = tests.get("TestLWorld::test84");

rerun_and_recompile_for("TestLWorld::test84", 10,
() -> { test84(src, dst);
Asserts.assertTrue(Arrays.equals(src, dst)); });

0 comments on commit 330f2f2

Please sign in to comment.