On Thu, 9 Jul 2026 14:59:00 GMT, Aleksey Shipilev <[email protected]> wrote:

>> Andrew Haley has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   More
>
> src/hotspot/share/ci/ciEnv.cpp line 1074:
> 
>> 1072:                                compiler, 
>> CompLevel(task()->comp_level()),
>> 1073:                                nmethod::Flags(has_unsafe_access, 
>> has_wide_vectors, has_monitors, has_scoped_access));
>> 1074:     {
> 
> I don't see why this move is needed?

It's another leftover.

Something that I have always hated: code is installed and run _before_ it is 
printed. So, if it crashes, too bad.

I should submit this fix as a separate PR.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/28541#discussion_r3557957480

Reply via email to