RFR: JDK-8283326: Implement SafeFetch statically
Thomas Stuefe
stuefe at openjdk.java.net
Mon Apr 11 09:21:41 UTC 2022
On Mon, 11 Apr 2022 06:24:05 GMT, David Holmes <dholmes at openjdk.org> wrote:
> Hi Thomas,
>
> This looks really good! Definite approval in principle. I can't comment on the details of the .S files. A few nits and comments. Main is comment is that SAFEFETCH_METHOD_x defines don't seem to actually be needed ??
>
> Thanks, David
Thank you David!
SAFEFETCH_METHOD_xx was supposed to be a clean separation between safefetch method and platform, since there is no clear 1:1 relation and you can, and may need to, mix and match. Examples:
- On Posix platforms you can use either static assembly or setjmp, and need to exclude one of them from compilation - either build safefetch_xx_xx.S, or build safefetch_sigjmp.cpp.
- On Windows, we use SEH, but I don't know yet if that works for Windows aarch64, and we still may need to switch to static assembly for this platform. I hope to avoid this though since MASM is a pain.
Therefore I'd like to keep the separation between SafeFetch method and platform, at least for now. Makes it easer to switch back and forth, and I find it actually cleaner.
Cheers Thomas
-------------
PR: https://git.openjdk.java.net/jdk/pull/7865
More information about the hotspot-dev
mailing list