On Thu, 13 Feb 2025 23:01:24 GMT, Dean Long <dl...@openjdk.org> wrote:
>> Vladimir Kozlov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> rename SA argument > > src/hotspot/share/runtime/vframe.inline.hpp line 178: > >> 176: INTPTR_FORMAT " not found or invalid at %d", >> 177: p2i(_frame.pc()), decode_offset); >> 178: nm()->print_on_v(&ss); > > I suggest removing _v suffix to reduce changes and match existing naming. Done. Testing now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23533#discussion_r1956985708
