On Tue, 18 Nov 2025 18:19:00 GMT, Frederic Parain <[email protected]> wrote:
> Add some helper methods to test properties of layouts in a more reliable way. > > Tested with Mach5, tier 1 to 3. > > Fred src/hotspot/share/oops/arrayKlass.cpp line 231: > 229: break; > 230: case LayoutKind::NULLABLE_ATOMIC_FLAT: > 231: props = ArrayKlass::ArrayProperties::NON_ATOMIC; Why does NULLABLE_ATOMIC_FLAT produce NON_ATOMIC? Something, maybe just naming?, seems off here. I know this is a refactoring of existing code and so has the same meaning but it doesn't make sense to me ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/1745#discussion_r2543646279
