On Mon, 9 Mar 2026 16:09:31 GMT, Damon Fenacci <[email protected]> wrote:
> it might be slightly better if you need to debug and slightly less risky with
> variable shadowing
These are good arguments. I've replaced the `RETURN_ON_BAILOUT` macro with two
regular methods in d16b85f8184.
> src/hotspot/share/opto/library_call.cpp line 862:
>
>> 860: if (stopped()) { \
>> 861: return true; \
>> 862: }
>
> Since you use the `while(0)` idiom I think it would make sense to move this
> inside as well.
Changed as requested in e889700699c.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2910388940
PR Review Comment: https://git.openjdk.org/jdk/pull/30139#discussion_r2910383346
