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