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