RFR: 8233885: Test fails with assert(comp != __null) failed: Ensure we have a compiler
Tobias Hartmann
tobias.hartmann at oracle.com
Tue Nov 12 08:36:59 UTC 2019
Hi Jie,
seems reasonable to me but Igor should have a look as well.
Thanks,
Tobias
On 11.11.19 14:28, Jie Fu wrote:
> Hi Tobias,
>
> Thank you for your review and valuable comments.
> Updated: http://cr.openjdk.java.net/~jiefu/8233885/webrev.02/
>
> Thanks a lot.
> Best regards,
> Jie
>
> On 2019/11/11 下午3:44, Tobias Hartmann wrote:
>> Hi Jie,
>>
>> what about the high-only-quick-internal mode?
> It's really a nice catch.
> Fixed. Thanks.
>
>
>> While looking at the fix for 8227003, I spotted a little typo here ("mininum"):
>> https://hg.openjdk.java.net/jdk/jdk/rev/b95bead30957#l6.8
>> Maybe you can fix that as well with your patch.
> Done.
>
More information about the hotspot-compiler-dev
mailing list