RFR: 8241502: C2: Migrate x86_64.ad to MacroAssembler [v9]

Vladimir Kozlov kvn at openjdk.java.net
Fri Apr 30 17:04:57 UTC 2021


On Thu, 29 Apr 2021 20:37:12 GMT, John Tortugo <github.com+2249648+JohnTortugo at openjdk.org> wrote:

>> Relates to: https://bugs.openjdk.java.net/browse/JDK-8241502
>> Tested on: Linux tier1, 2 and 3
>> 
>> Can you please take a look whether these changes are going in the direction expected or not? If it is, I'll continue working on the `JDK-8241502` but I'd like to split it in a few PRs since it's a lot of changes.
>
> John Tortugo has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Encode cdql/cdqq using MacroAssembler.

Preliminary result shows compiler/c2/cr6340864/TestLongVect.java test failure on linux and windows -x64 when run with not G1 gc (Serial, Parallel, ZGC) (the line could be different again due to use of latest source base):

#  Internal Error (/workspace/open/src/hotspot/cpu/x86/assembler_x86.cpp:10753), pid=9483, tid=9495
#  assert(isByte(imm8)) failed: not a byte
#
# JRE version: Java(TM) SE Runtime Environment (17.0) (fastdebug build 17-internal+0-LTS-2021-04-30-0438565.jdkgit)
# Java VM: Java HotSpot(TM) 64-Bit Server VM (fastdebug 17-internal+0-LTS-2021-04-30-0438565.jdkgit, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, parallel gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x6471b4]  Assembler::sarq(RegisterImpl*, int)+0x164

V  [libjvm.so+0x6471b4]  Assembler::sarq(RegisterImpl*, int)+0x164
V  [libjvm.so+0x34d566]  sarL_rReg_immNode::emit(CodeBuffer&, PhaseRegAlloc*) const+0xb6
V  [libjvm.so+0x1588551]  PhaseOutput::scratch_emit_size(Node const*)+0x421
V  [libjvm.so+0x157f3f8]  PhaseOutput::shorten_branches(unsigned int*)+0x2b8
V  [libjvm.so+0x159165a]  PhaseOutput::Output()+0xcca
V  [libjvm.so+0xa0636b]  Compile::Code_Gen()+0x43b

-------------

PR: https://git.openjdk.java.net/jdk/pull/2420


More information about the hotspot-compiler-dev mailing list