RFR: 8265480: add basic JVMCI support for JEP 309: Dynamic Class-File Constants [v2]
Doug Simon
dnsimon at openjdk.java.net
Thu Apr 22 10:32:24 UTC 2021
On Wed, 21 Apr 2021 21:44:51 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:
>> No - it should not be there at all: https://github.com/openjdk/jdk/pull/3578/commits/b70c7b1a64d51fce23edde860f04749e8e49d34f
>> Thanks for spotting.
>
> Oops, i think i need glasses, the switch has break statements that results in processing of the `raw_value`:
> ```c++
> switch (bt2) {
> case T_BOOLEAN: raw_value = value.z; break;
> ...
I need more than glasses since I wrote this code! Thankfully the test I added also showed the problem with removing this "unreachable" code.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3578
More information about the hotspot-compiler-dev
mailing list