RFR(sh/jdk11): Shenandoah: Isolate shared-code changes

Roman Kennke rkennke at redhat.com
Wed Jul 8 10:24:49 UTC 2020


On Wed, 2020-07-08 at 11:57 +0200, Roland Westrelin wrote:
> > https://cr.openjdk.java.net/~rkennke/shenandoah-jdk11u-upstream/webrev.05-shared/src/hotspot/share/opto/ifnode.cpp.udiff.html
> 
> Actually we can put:
> 
> (req() == 3 && dom->in(2) != in(2)) || // Not same input 2?
> 
> back in and have no diff with upstream. It's dead code so it doesn't
> matter if it's there or not.

Ok, so I did:

http://cr.openjdk.java.net/~rkennke/shjdk11-isolate-shared/webrev.01/

Thanks for reviewing!

I still need a review of the rest of the patch.

Roman



More information about the shenandoah-dev mailing list