On Sat, 8 Jul 2023 10:48:15 GMT, sid8606 <d...@openjdk.org> wrote: >> Implementation of "Foreign Function & Memory API" for s390x (Big Endian). > > sid8606 has updated the pull request incrementally with one additional commit > since the last revision: > > Address suggestions from Jorn Vernee
src/hotspot/cpu/s390/downcallLinker_s390.cpp line 162: > 160: > 161: assert(!_needs_return_buffer, "unexpected needs_return_buffer"); > 162: bool should_save_return_value = _needs_transition;; This should always be `true`, so I don't think you need the `if` statements around the spill/fill code below. See: https://github.com/openjdk/jdk/pull/15025 (`should_save_return_value` being dependent on `_needs_transition` is a bug). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14801#discussion_r1274138222