stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8321122: Shenandoah: Remove ShenandoahLoopOptsAfterExpansion flag

Browse Source 
Reviewed-by: kdnilsen, roland, rkennke
This commit is contained in:
Aleksey Shipilev 2023-12-06 21:19:24 +00:00
parent 2830dd2a7d
commit b02fc8681e
2 changed files with 7 additions and 13 deletions
src/hotspot/share/gc/shenandoah
c2
shenandoahSupport.cpp
shenandoah_globals.hpp

17
src/hotspot/share/gc/shenandoah/c2/shenandoahSupport.cpp

@ -50,21 +50,18 @@ bool ShenandoahBarrierC2Support::expand(Compile* C, PhaseIterGVN& igvn) {
state->load_reference_barriers_count()) > 0) {
assert(C->post_loop_opts_phase(), "no loop opts allowed");
C->reset_post_loop_opts_phase(); // ... but we know what we are doing
bool attempt_more_loopopts = ShenandoahLoopOptsAfterExpansion;
C->clear_major_progress();
PhaseIdealLoop::optimize(igvn, LoopOptsShenandoahExpand);
if (C->failing()) return false;
PhaseIdealLoop::verify(igvn);
if (attempt_more_loopopts) {
C->set_major_progress();
if (!C->optimize_loops(igvn, LoopOptsShenandoahPostExpand)) {
return false;
}
C->clear_major_progress();
C->process_for_post_loop_opts_igvn(igvn);
if (C->failing()) return false;
C->set_major_progress();
if (!C->optimize_loops(igvn, LoopOptsShenandoahPostExpand)) {
return false;
}
C->clear_major_progress();
C->process_for_post_loop_opts_igvn(igvn);
if (C->failing()) return false;
C->set_post_loop_opts_phase(); // now for real!
}
return true;

3
src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp

@ -352,9 +352,6 @@
develop(bool, ShenandoahVerifyOptoBarriers, trueInDebug, \
"Verify no missing barriers in C2.") \
\
product(bool, ShenandoahLoopOptsAfterExpansion, true, DIAGNOSTIC, \
"Attempt more loop opts after barrier expansion.") \
\
// end of GC_SHENANDOAH_FLAGS