-
Notifications
You must be signed in to change notification settings - Fork 108
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
Merge jdk #846
Merge jdk #846
Conversation
… modes Reviewed-by: fyang, yadongwang
…orrectly Reviewed-by: fyang, yadongwang
Reviewed-by: rcastanedalo, thartmann
Reviewed-by: alanb, dholmes
Reviewed-by: stuefe, dholmes
Reviewed-by: thartmann
Reviewed-by: coleenp, dnsimon
Reviewed-by: kvn, never
Reviewed-by: kbarrett, dholmes
Reviewed-by: stefank, tschatzl
Reviewed-by: iklam
Reviewed-by: jlahoda
Reviewed-by: fparain, kbarrett
Reviewed-by: lfoltan
…n methods inside other annotations Reviewed-by: jlahoda
…tion Reviewed-by: alanb
Reviewed-by: tsteele, dholmes
Co-authored-by: Calvin Cheung <ccheung@openjdk.org> Reviewed-by: lmesnik, ccheung
Reviewed-by: dfuchs, sspitsyn, cjplummer
Reviewed-by: stefank, tschatzl
Reviewed-by: stefank, kbarrett
Reviewed-by: coleenp, dholmes
…ld succeed Reviewed-by: kvn, thartmann
…was added although the intrinsic bailed out Reviewed-by: thartmann, kvn
Reviewed-by: chagedorn, kvn, jbhateja, vlivanov
Reviewed-by: kvn, thartmann
Reviewed-by: mullan
Reviewed-by: rhalade
Reviewed-by: hannesw
👋 Welcome back dsimms! A progress list of the required criteria for merging this PR into |
Added tag jdk-21+8 for changeset bc750f7
Added tag jdk-21+9 for changeset 638d612
/summary Merge from jdk-21+5 to jdk-21+10 |
@MrSimms This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be:
You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been no new commits pushed to the ➡️ To integrate this PR with the above commit message to the |
@MrSimms Setting summary to |
/integrate |
Going to push as commit fc300fa. |
Webrevs |
Merge from jdk-21+5 to jdk-21+10
Progress
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/846/head:pull/846
$ git checkout pull/846
Update a local copy of the PR:
$ git checkout pull/846
$ git pull https://git.openjdk.org/valhalla.git pull/846/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 846
View PR using the GUI difftool:
$ git pr show -t 846
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/846.diff