[jdk16] Integrated: 8258790: C2: Crash on empty macro node list

Tue Dec 22 12:15:00 UTC 2020

On Mon, 21 Dec 2020 19:22:52 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

> The fix for 8257624 is incomplete: it doesn't cover the case when macro node array becomes empty as a result of multiple nodes removal. In such case, the out-of-bounds access still takes place (at index "-1"). 
> 
> Proposed fix is to adjust the index before performing "i > 0" check.    
> 
> Testing:
> - [x] failing tests
> - [ ] hs-precheckin-comp, hs-tier1, hs-tier2

This pull request has now been integrated.

Changeset: 772addfd
Author:    Vladimir Ivanov <vlivanov at openjdk.org>
URL:       https://git.openjdk.java.net/jdk16/commit/772addfd
Stats:     10 lines in 1 file changed: 0 ins; 6 del; 4 mod

8258790: C2: Crash on empty macro node list

Reviewed-by: kvn, chagedorn

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

PR: https://git.openjdk.java.net/jdk16/pull/56