RFR: 8347718: Unexpected NullPointerException in C2 compiled code due to ReduceAllocationMerges

[Vladimir Kozlov]

On Mon, 27 Jan 2025 03:36:24 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

> Please, review this patch to fix an issue in reducing allocation merges used by Cmp nodes. The attached test case, adapted from the one attached to the JBS issue, can be used to reproduce the issue. The root of the problem was not properly checking the return value of `optimize_ptr_compare` against different Cmp mask predicates.
> 
> I tested this on OSX(ARM64) & Linux(Intel) tier1-4.

test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndPointerComparisons.java line 33:

> 31:  *                   -XX:-TieredCompilation -Xcomp -server
> 32:  *                   compiler.c2.TestReduceAllocationAndPointerComparisons
> 33:  * @run main/othervm compiler.c2.TestReduceAllocationAndPointerComparisons

You don't need `/othervm` if you not use VM's flags.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23312#discussion_r1930839110