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