RFR: JDK-8310425: [JVMCI] compiler/runtime/TestConstantDynamic: lookupConstant returned an object of incorrect type: null

Tom Rodriguez never at openjdk.org
Mon Jun 26 16:47:03 UTC 2023


On Wed, 21 Jun 2023 10:26:19 GMT, David Leopoldseder <davleopo at openjdk.org> wrote:

> Fix JVMCI handling of null dynamic constants and dynamic constant errors.
> 
> For null dynamic constants, the JVMCI code was checking for `Universe::the_null_sentinel` but `ConstantPool::resolve_possibly_cached_constant_at` returns `nullptr` for null constants.
> 
> In the case of errors thrown by a bootstrap method, `ConstantPool::resolve_possibly_cached_constant_at` already propagates the `BootstrapMethodError` so there's no need to check for `tag.is_dynamic_constant_in_error()`.

Marked as reviewed by never (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/14582#pullrequestreview-1498969428


More information about the hotspot-compiler-dev mailing list