RFR: one more missing UseShenandoahGC check in 8u

Aleksey Shipilev shade at redhat.com
Wed Oct 4 11:52:52 UTC 2017


On 10/04/2017 01:45 PM, Roland Westrelin wrote:
> http://cr.openjdk.java.net/~roland/shenandoah/8u_useshenandoahgc3/webrev.00/

Okay. Seems cleaner just to split the paths?

 Node zlen_arg = NULL;
 if (UseShenandoahGC) {
   Node *cast = new (C) CastPPNode(z, TypePtr::NOTNULL);
   cast->init_req(0, control());
   _gvn.set_type(cast, cast->bottom_type());
   C->record_for_igvn(cast);
   zlen_arg = load_array_length(cast);
 } else {
   zlen_arg = load_array_length(z);
 }

Thanks,
-Aleksey



More information about the shenandoah-dev mailing list