RFR: one more missing UseShenandoahGC check in 8u

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


On 10/04/2017 01:56 PM, Roland Westrelin wrote:
>> 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);
>>  }
> 
> Ok. Should I wait for more reviews or can I push it?

This is a trivial change, run hotspot_gc_shenandoah with it, and push.

-Aleksey




More information about the shenandoah-dev mailing list