On Fri, 30 Jan 2026 09:14:56 GMT, Quan Anh Mai <[email protected]> wrote:

>> Hi,
>> 
>> This PR implements the support of `NULLABLE_NON_ATOMIC_FLAT` layout in the 
>> JITs. There is nothing to do in C2. In C1, I need to implement 
>> loading/storing nullable value of a non-atomic field.
>> 
>> The test `TestValueClasses` is failing with `-XX:-TieredCompilation`, which 
>> I suspect is due to the substitutability test handling padding bytes 
>> incorrectly.
>> 
>> Please kindly review what there are for now, thanks a lot.
>
> Quan Anh Mai has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   add assert

Marked as reviewed by thartmann (Committer).

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

PR Review: 
https://git.openjdk.org/valhalla/pull/1970#pullrequestreview-3728103112

Reply via email to