RFR: 8338924: C1: assert(0 <= i && i < _len) failed: illegal index 5 for length 5

[Coleen Phillimore]

On Tue, 27 Aug 2024 18:01:16 GMT, Matias Saavedra Silva <matsaave at openjdk.org> wrote:

> The change in [JDK-8335664](https://bugs.openjdk.org/browse/JDK-8335664) caused a crash when initializing basic blocks with `-Xcomp`. This change introduces a check to see if JSR is the last bytecode in its method so that expected behavior matches the previous patch. Verified with tier 1-6 tests.

Just a comment.

test/hotspot/jtreg/runtime/interpreter/LastJsrTest.java line 39:

> 37: public class LastJsrTest {
> 38:     public static void main(String[] args) {
> 39:         for (int i = 0; i < 1000; ++i) {

Don't you need 10,000 in your loop to trigger compilation?

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

PR Review: https://git.openjdk.org/jdk/pull/20732#pullrequestreview-2273143025
PR Review Comment: https://git.openjdk.org/jdk/pull/20732#discussion_r1739172439