> The lworld branch has support for null-restricted arrays, and to support this > code has been adjacent to the code that perform array copy check casts. The > code that performs the check casts used to be located inside the GC access > barriers, but the code that finally threw exceptions were located in the > runtime code. In the lworld branch the code that throws the check cast > exception and the new null pointer exception has been moved into the GC > access barriers. > > I propose that we revert back to the old style where the exception throwing > was somewhat decoupled from the GC access barriers.
Stefan Karlsson has updated the pull request incrementally with two additional commits since the last revision: - Mark oop_arraycopy as nodiscard - Terminate OopCopyResult in oop_arraycopy_raw ------------- Changes: - all: https://git.openjdk.org/valhalla/pull/1792/files - new: https://git.openjdk.org/valhalla/pull/1792/files/5b4063d5..9894b597 Webrevs: - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1792&range=01 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1792&range=00-01 Stats: 12 lines in 2 files changed: 7 ins; 0 del; 5 mod Patch: https://git.openjdk.org/valhalla/pull/1792.diff Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1792/head:pull/1792 PR: https://git.openjdk.org/valhalla/pull/1792
