[jdk16] Integrated: 8259061: C2: assert(found) failed: memory-writing node is not placed in its original loop or an ancestor of it

[Roberto Castañeda Lozano]

On Mon, 11 Jan 2021 12:27:18 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

> Remove assertion that is too general, that is, it can fail on compilations where C2 generates correct code otherwise.
> 
> The assertion was introduced in [JDK-8255763](https://bugs.openjdk.java.net/browse/JDK-8255763) to prevent potential miscompilations in the case of irreducible CFGs, and will be reconsidered in [JDK-8258894](https://bugs.openjdk.java.net/browse/JDK-8258894) where the general case is addressed.
> 
> Tested by building (release and debug) on different platforms.

This pull request has now been integrated.

Changeset: 417e1d1a
Author:    Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk16/commit/417e1d1a
Stats:     20 lines in 1 file changed: 0 ins; 20 del; 0 mod

8259061: C2: assert(found) failed: memory-writing node is not placed in its original loop or an ancestor of it

Remove assertion that is too general, that is, it can fail on compilations where
C2 generates correct code otherwise.

Reviewed-by: chagedorn, thartmann, kvn

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

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