RFR: 8256924: ppc, ppcle, s390: JVM crashes at VM init after JDK-8254231

Aleksey Shipilev shade at openjdk.java.net
Tue Nov 24 16:14:57 UTC 2020


On Tue, 24 Nov 2020 16:03:33 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> src/hotspot/cpu/ppc/methodHandles_ppc.cpp line 319:
>> 
>>> 317:     if (iid == vmIntrinsics::_linkToNative) {
>>> 318:       assert(for_compiler_entry, "only compiler entry is supported");
>>> 319:     }
>> 
>> I believe it is customary to write conditional asserts like:
>>   assert(for_compiler_entry || iid != vmIntrinsics::_linkToNative, "only compiler entry is supported for linkToNative")
>> 
>> ...same thing for S390.
>
> Thanks for reviewing it!
> I've copied the code from x86 and I'd prefer to keep it consistent even though I like your version more. Are you ok with it?

Yeah, that's fine. I don't have a strong opinion here.

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

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


More information about the hotspot-compiler-dev mailing list