> 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 two additional commits since the last revision: - no_null_marker - rename null_free in InlineTypeNode::initialize_fields to has_null_marker ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/1970/files - new: https://git.openjdk.org/valhalla/pull/1970/files/bddd1516..3fb86af0 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1970&range=02 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1970&range=01-02 Stats: 4 lines in 2 files changed: 0 ins; 0 del; 4 mod Patch: https://git.openjdk.org/valhalla/pull/1970.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1970/head:pull/1970 PR: https://git.openjdk.org/valhalla/pull/1970
