Integrated: 8252496: C2: Useless code in MergeMemNode::Ideal

Emanuel Peter duke at openjdk.java.net
Thu Feb 10 09:33:09 UTC 2022


On Wed, 9 Feb 2022 09:23:50 GMT, Emanuel Peter <duke at openjdk.java.net> wrote:

> This code was there since `initial load`. At that point there was other code that still depended on this code.
> The other code was removed [here](https://github.com/eme64/jdk/commit/79ad5512cfe5c4ef7d0b61616f3763c62d3427a5). This was the change set for [JDK-8243670](https://bugs.openjdk.java.net/browse/JDK-8243670).
> Our code was left behind.
> 
> I removed the useless code, and ran tests to verify that nothing broke due to side-effects.

This pull request has now been integrated.

Changeset: fa0a72c0
Author:    Emanuel Peter <emanuel.peter at oracle.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/fa0a72c030432f9ea4ad9913a2bb4096324410aa
Stats:     24 lines in 1 file changed: 0 ins; 24 del; 0 mod

8252496: C2: Useless code in MergeMemNode::Ideal

Reviewed-by: thartmann, chagedorn, vlivanov

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

PR: https://git.openjdk.java.net/jdk/pull/7401


More information about the hotspot-compiler-dev mailing list