[jdk17u-dev] RFR: 8283326: Implement SafeFetch statically [v2]

[Sergey Nazarkin]

On Sat, 8 Jul 2023 09:25:45 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Sergey Nazarkin has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
>> 
>>  - RiscV part
>>  - Fix builds without precompiled headers
>>  - Backport bdf8a2a2050393e91800786f8d5a5d6805f936eb
>
> src/hotspot/os/posix/vmError_posix.cpp line 90:
> 
>> 88:     }
>> 89:   }
>> 90: #endif // CAN_SHOW_REGISTERS_ON_ASSERT
> 
> Above hunks are not part of the patch, and should not be needed for safefetch. Why are they here?

It's here to keep existing logic of this hunk. And to sync with mainline JDK version. The file was previously modified in scope of JDK-8289350 that doesn't affect JDK19+.

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

PR Review Comment: https://git.openjdk.org/jdk17u-dev/pull/1557#discussion_r1261320337