RFR: Remove ShenandoahCompileCheck
Aleksey Shipilev
shade at redhat.com
Thu Oct 11 16:49:28 UTC 2018
This increases our upstream exposure. I asked if upstream wants to have it generically:
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2018-October/030912.html
...but there is no need to have it in upstreamable Shenandoah:
diff -r 429af28fcb53 src/hotspot/share/compiler/compileBroker.cpp
--- a/src/hotspot/share/compiler/compileBroker.cpp Thu Oct 11 09:40:48 2018 -0400
+++ b/src/hotspot/share/compiler/compileBroker.cpp Thu Oct 11 18:47:55 2018 +0200
@@ -2089,8 +2089,4 @@
ciMethod* target = ci_env.get_method_from_handle(target_handle);
-#if INCLUDE_SHENANDOAHGC
- bool target_compilable = target->can_be_parsed() && target->can_be_compiled();
-#endif
-
TraceTime t1("compilation", &time);
EventCompilation event;
@@ -2124,11 +2120,4 @@
}
-#if INCLUDE_SHENANDOAHGC
- guarantee(!UseShenandoahGC || !ShenandoahCompileCheck || !target_compilable || (compilable !=
ciEnv::MethodCompilable_not_at_tier),
- "Not compilable on level %d due to: %s", task_level, failure_reason);
- guarantee(!UseShenandoahGC || !ShenandoahCompileCheck || !target_compilable ||(compilable !=
ciEnv::MethodCompilable_never || !target_compilable),
- "Never compilable due to: %s", failure_reason);
-#endif
-
post_compile(thread, task, !ci_env.failing(), &ci_env);
if (event.should_commit()) {
diff -r 429af28fcb53 src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp
--- a/src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp Thu Oct 11 09:40:48 2018 -0400
+++ b/src/hotspot/share/gc/shenandoah/shenandoah_globals.hpp Thu Oct 11 18:47:55 2018 +0200
@@ -420,7 +420,4 @@
"Use Elastic TLABs with Shenandoah") \
\
- diagnostic(bool, ShenandoahCompileCheck, false, \
- "Assert that methods are successfully compilable") \
- \
experimental(bool, ShenandoahAlwaysClearSoftRefs, false, \
"Clear soft references unconditionally") \
Testing: tier3_gc_shenandoah
Thanks,
-Aleksey
More information about the shenandoah-dev
mailing list