RFR: 8289856: [PPC64] SIGSEGV in C2Compiler::init_c2_runtime() after JDK-8289060 [v2]

Martin Doerr mdoerr at openjdk.org
Thu Jul 7 07:59:17 UTC 2022


On Thu, 7 Jul 2022 00:16:18 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use VMRegImpl::Bad() for vector registers and remove null check again.
>
> src/hotspot/share/opto/c2compiler.cpp line 69:
> 
>> 67:   for (OptoReg::Name i=OptoReg::Name(0); i<OptoReg::Name(REG_COUNT); i = OptoReg::add(i,1)) {
>> 68:     VMReg r = OptoReg::as_VMReg(i);
>> 69:     if (r != nullptr && r->is_valid()) {
> 
> Instead of changing shared code, how about changing ppc.ad to use VMRegImpl::Bad() instead of NULL?

That makes more sense. I've changed it. Thanks!

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

PR: https://git.openjdk.org/jdk/pull/9403


More information about the hotspot-compiler-dev mailing list