RFR (11): [backport] 8221766: Load-reference barriers for Shenandoah
Roland Westrelin
rwestrel at redhat.com
Wed Jun 5 12:25:44 UTC 2019
> It reverts the part back to original jdk11 upstream because we no longer
> need to insert into the control flow there. Is that not ok?
Ah ok. That change is good.
>> cfgnode.hpp
>>
>> That change doesn't seem needed
>
> Are you sure? We do have a corresponding call to step_over_gc_barrier()
> in jdk/jdk too.
>
> http://hg.openjdk.java.net/jdk/jdk/file/ce8bab2c4185/src/hotspot/share/opto/cfgnode.cpp#l1451
cfgnode.*h*pp
The following change which removes:
306 Node* dominated_by(Node* prev_dom, PhaseIterGVN* igvn);
and adds it back:
393 Node* dominated_by(Node* prev_dom, PhaseIterGVN* igvn);
Roland.
More information about the shenandoah-dev
mailing list