RFR: 8259316: [REDO] C1/C2 compiler support for blackholes
Aleksey Shipilev
shade at openjdk.java.net
Wed Mar 31 15:47:27 UTC 2021
On Thu, 25 Mar 2021 15:40:59 GMT, Doug Simon <dnsimon at openjdk.org> wrote:
>> The way C1 and C2 see this as synthetic intrinsic is modeled after `vmIntrinsics::_compiledLambdaForm`. Unfortunately, `Method::init_intrinsic_id` does not have access to `CompilerOracle` (which is solvable), and current placement in `ciMethod` constructor guarantees the method is resolved. I wonder if JVMCI would benefit from the direct `shouldBlackholeMethod`, like it already has `shouldInlineMethod`?
>
> I think it would be better in `JVMCIEnv::get_jvmci_method` which is the JVMCI equivalent of `ciMethod::ciMethod`.
`Method::init_intrinsic_id` does not have access to `CompilerOracle` and it also does not check for method resolution (I think). I added the similar block to `JVMCIEnv::get_jvmci_method` as @dougxc suggested.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2024
More information about the hotspot-dev
mailing list