RFR: 8151723: [JVMCI] JVMCIRuntime::treat_as_trivial: Don't limit trivial prefixes to boot class path
Doug Simon
doug.simon at oracle.com
Thu Mar 17 23:01:22 UTC 2016
Looks good.
> On 17 Mar 2016, at 18:30, Christian Thalinger <christian.thalinger at oracle.com> wrote:
>
> https://bugs.openjdk.java.net/browse/JDK-8151723
>
> There's no guarantee that a JVMCI compiler will be loaded from the boot class path. The mechanism provided for a JVMCI compiler to communicate to the compiler broker that it doesn't want to compile itself should take this into account.
>
> diff -r 3256d4204291 src/share/vm/jvmci/jvmciRuntime.cpp
> --- a/src/share/vm/jvmci/jvmciRuntime.cpp Wed Mar 16 10:45:43 2016 +0100
> +++ b/src/share/vm/jvmci/jvmciRuntime.cpp Thu Mar 17 07:29:04 2016 -1000
> @@ -800,12 +800,9 @@ void JVMCIRuntime::shutdown(TRAPS) {
>
> bool JVMCIRuntime::treat_as_trivial(Method* method) {
> if (_HotSpotJVMCIRuntime_initialized) {
> - oop loader = method->method_holder()->class_loader();
> - if (loader == NULL) {
> - for (int i = 0; i < _trivial_prefixes_count; i++) {
> - if (method->method_holder()->name()->starts_with(_trivial_prefixes[i])) {
> - return true;
> - }
> + for (int i = 0; i < _trivial_prefixes_count; i++) {
> + if (method->method_holder()->name()->starts_with(_trivial_prefixes[i])) {
> + return true;
> }
> }
> }
>
More information about the hotspot-compiler-dev
mailing list