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