On Tue, 5 May 2026 08:33:11 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 incrementally with one additional > commit since the last revision: > > removed unneeded modules LGTM, thanks. ------------- Marked as reviewed by phubner (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2383#pullrequestreview-4234722350
