RFR: 8265480: add basic JVMCI support for JEP 309: Dynamic Class-File Constants [v2]

Paul Sandoz psandoz at openjdk.java.net
Wed Apr 21 20:26:10 UTC 2021


On Wed, 21 Apr 2021 19:57:05 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> This PR adds support to JVMCI for CONSTANT_Dynamic constant pool entries. It brings the JVMCI support on par with the support in CI for C1 and C2 with one difference: JVMCI will resolve unresolved condy entries where as CI bails out in this case (it expects condy entries to be resolved by the interpreter).
>> 
>> When using JVMCI for JIT compilation, unresolved entries should only be seen when running under -Xcomp.
>> 
>> When using JVMCI for AoT (e.g. Native Image), most entries will be unresolved since no other code apart from the AoT compiler is running.
>
> Doug Simon has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixed date in copyright headers

src/hotspot/share/jvmci/jvmciCompilerToVM.cpp line 659:

> 657:       }
> 658: 
> 659:       JVMCIObject result = JVMCIENV->call_PrimitiveConstant_forTypeChar(type2char(bt2), raw_value, JVMCI_CHECK_NULL);

Given the above switch is total this part is unreachable? Did you mean for it to occur after the `if (!is_reference_type(bt)) ` block?

-------------

PR: https://git.openjdk.java.net/jdk/pull/3578


More information about the hotspot-compiler-dev mailing list