On Mon, 15 Jun 2026 09:25:37 GMT, Casper Norrbin <[email protected]> wrote:
>> Hi everyone, >> >> The `objArrayOop` gtest currently only tests reference arrays, but should >> also test on flat arrays. This is reflected by a `FIXME` at the top of the >> file. >> >> For this change I expanded the test to also cover flat arrays. The original >> tested options for ref arrays (mostly related to header size and layout) >> make less sense for a flattened array, so I instead picked more relevant >> options to test across. I also refactored the whole test so it becomes more >> readable. The actual data tested on remains untouched. >> >> The jtreg runner was also missing a couple of configurations, had a bad >> gtest specifier, and had a flag misspelled, so I fixed all those as well. >> >> Testing: >> - Tier 1 >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Casper Norrbin has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains four additional > commits since the last revision: > > - feedback fixes > - Merge branch 'lworld' into test_objArrayOop-flatarrays > - removed unneeded modules > - expanded test to cover flatarrays Thank you for the reviews. It's time we get this in. ------------- PR Comment: https://git.openjdk.org/valhalla/pull/2383#issuecomment-4789479424
