[11u] RFR (XS) 8253353: Crash in C2: guarantee(n != NULL) failed: No Node
Vladimir Kozlov
vladimir.kozlov at oracle.com
Mon Feb 8 17:03:01 UTC 2021
I am fine with keeping empty line.
Thanks,
Vladimir K
On 2/8/21 2:33 AM, Aleksey Shipilev wrote:
> On 2/4/21 9:21 AM, Aleksey Shipilev wrote:
>> Original bugfix:
>> https://bugs.openjdk.java.net/browse/JDK-8253353
>> https://git.openjdk.java.net/jdk16/commit/1926765f
>>
>> This fixes the regression introduced by:
>> https://bugs.openjdk.java.net/browse/JDK-8240576
>>
>> ...which was already backported to 11.0.8.
>>
>> The patch is simple, but it fails to apply cleanly, because one of the asserts in loopnode.cpp
>> cannot be placed automatically. I placed near "loop->_nest++" in 11u code.
>>
>> 11u variant:
>> https://cr.openjdk.java.net/~shade/8253353/webrev.11u.01
>>
>> Testing: tier{1,2,3}; new test fails without the patch, passes with it
>
> I see Vladimir's review here (but not in my mail, because jdk-updates-dev mailing list is temporarily broken):
> https://mail.openjdk.java.net/pipermail/jdk-updates-dev/2021-February/004864.html
>
> Vladimir says:
> > It will match our backport exactly if you remove empty line after first assert().
>
> ...and I would prefer to keep it with the original empty line in the code.
>
More information about the jdk-updates-dev
mailing list