-
Notifications
You must be signed in to change notification settings - Fork 5.8k
8354674: AArch64: Intrinsify Unsafe::setMemory #25147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
👋 Welcome back aph! A progress list of the required criteria for merging this PR into |
❗ This change is not yet ready to be integrated. |
@theRealAph The following labels will be automatically applied to this pull request:
When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing lists. If you would like to change these labels, use the /label pull request command. |
Webrevs
|
Apple M1, small memory blocks:
|
Graviton 4, small memory blocks:
|
Apple M1:
|
Graviton 4:
|
@@ -0,0 +1,93 @@ | |||
/* | |||
* Copyright (c) 2024, Oracle and/or its affiliates. All rights reserved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2025 maybe more suitable
Looking at the improvements made, I suggest we also change (in
to
|
This intrinsic is generally faster than the current implementation for Panama segment operations for all writes larger than about 8 bytes in size, increasing to more than 2* the performance on larger memory blocks on Graviton 2, between "panama" (C2 generated, what we use now) and "unsafe" (this intrinsic).
Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/25147/head:pull/25147
$ git checkout pull/25147
Update a local copy of the PR:
$ git checkout pull/25147
$ git pull https://git.openjdk.org/jdk.git pull/25147/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 25147
View PR using the GUI difftool:
$ git pr show -t 25147
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/25147.diff
Using Webrev
Link to Webrev Comment