RFR: 8315771: [JVMCI] Resolution of bootstrap methods with int[] static arguments [v5]

Sacha Coppey duke at openjdk.org
Wed Sep 13 14:15:40 UTC 2023


On Wed, 13 Sep 2023 13:53:10 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> After double checking, there is in fact already an `assert` for checking if the index is out of bounds. Should I change it to an exception?
>
> Is the assertion in code that can throw exceptions?

The assertion is [here](https://github.com/openjdk/jdk/blob/bd477810b176696e0fd043f5594663ebcf9884cf/src/hotspot/share/oops/constantPool.hpp#L621-L625). I don't think we can throw exception there. I could check the size in `jvmciCompilerToVM.bootstrapArgumentIndexAt`, but it would require a bit of code duplication to obtain it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15588#discussion_r1324582677


More information about the hotspot-compiler-dev mailing list