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