-
Notifications
You must be signed in to change notification settings - Fork 121
Merge jdk #1516
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 #1516
Conversation
…witching works as expected Co-authored-by: Christian Hagedorn <[email protected]> Reviewed-by: chagedorn, epeter
Reviewed-by: erikj, chagedorn
…or not zero in counted loop Reviewed-by: chagedorn, epeter, qamai
…pe() Reviewed-by: rcastanedalo, thartmann
…dNode with LShift operand Reviewed-by: epeter, thartmann
…W_C_FUNCTION Reviewed-by: tschatzl, jsjolen
Reviewed-by: wetmore, hchao
Reviewed-by: dholmes
Reviewed-by: rcastanedalo, chagedorn, thartmann
Reviewed-by: stefank, lkorinth, stuefe
…util.concurrent Reviewed-by: pminborg, dl, rgiulietti, alanb
Reviewed-by: cjplummer, lmesnik
Reviewed-by: erikj
…k on macOS Reviewed-by: prr, psadhukhan
…rigger assert Reviewed-by: fyang, fjiang
Reviewed-by: fyang, luhenry
…ase superclass Reviewed-by: lmesnik, amenkov
…unication in nsk/jdi tests Reviewed-by: lmesnik, amenkov
… in tier6 Reviewed-by: dfuchs, smarks
Reviewed-by: wkemper, phh
Co-authored-by: Vladimir Ivanov <[email protected]> Reviewed-by: vlivanov, epeter
…l JDK-8355574 is fixed Reviewed-by: chagedorn
Reviewed-by: stefank, lucy
Reviewed-by: chagedorn, coleenp, jwaters
Reviewed-by: thartmann, chagedorn
Reviewed-by: shade, alanb
Reviewed-by: liach, hannesw
Reviewed-by: dfuchs
Reviewed-by: coleenp, lmesnik
…-Not instruction Reviewed-by: fjiang, fyang
Reviewed-by: rriggs, pminborg, naoto
…f it is mtNone Co-authored-by: Stefan Karlsson <[email protected]> Reviewed-by: stefank, coleenp, jsjolen
Co-authored-by: Ioi Lam <[email protected]> Reviewed-by: liach, eosterlund
Reviewed-by: mullan
Reviewed-by: shade, erikj
Reviewed-by: coleenp
Reviewed-by: vromero
…ection code Reviewed-by: jvernee
…ClassPrepareEvent for the debuggee main class Reviewed-by: sspitsyn, amenkov
…o lworld_merge_jdk_25_21
Added tag jdk-25+21 for changeset e2ae50d
/summary Merge jdk-25+21 |
👋 Welcome back dsimms! A progress list of the required criteria for merging this PR into |
@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 4 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details. ➡️ To integrate this PR with the above commit message to the |
@MrSimms Setting summary to |
Webrevs
|
/integrate |
Going to push as commit 899efed.
Your commit was automatically rebased without conflicts. |
Merge tag 'jdk-25+21' into lworld_merge_jdk_25_21
Added tag jdk-25+21 for changeset e2ae50d
Progress
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/1516/head:pull/1516
$ git checkout pull/1516
Update a local copy of the PR:
$ git checkout pull/1516
$ git pull https://git.openjdk.org/valhalla.git pull/1516/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 1516
View PR using the GUI difftool:
$ git pr show -t 1516
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/1516.diff
Using Webrev
Link to Webrev Comment