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