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

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


On Wed, 9 Feb 2022 12:31:51 GMT, Vladimir Ivanov <vlivanov at openjdk.org> 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.
>
> Looks good.

@iwanowww @chhagedorn @TobiHartmann thanks for the reviews!

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

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


More information about the hotspot-compiler-dev mailing list