-
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 #850
Merge jdk #850
Conversation
…6ToFloat yields different result than the interpreter Reviewed-by: sviswanathan, jbhateja, vlivanov
… wrongly split thru phi Reviewed-by: kvn, thartmann
Co-authored-by: Emanuel Peter <epeter@openjdk.org> Reviewed-by: epeter, kvn
…en IPv6 enabled Reviewed-by: djelinski, michaelm
…ll in early potential CHECK_NULL return Reviewed-by: weijun
…ication type Reviewed-by: ayang, tschatzl
Reviewed-by: thartmann
Reviewed-by: tschatzl, iwalulya
Reviewed-by: mcimadamore
Reviewed-by: ihse, psandoz, mcimadamore
… com/sun/jdi tests Reviewed-by: cjplummer, sspitsyn
Reviewed-by: erikj
Reviewed-by: ayang, iwalulya
Reviewed-by: mchung
Reviewed-by: cjplummer, sspitsyn
Reviewed-by: jjg
Reviewed-by: kbarrett, kvn
Reviewed-by: kvn, jwilhelm
…nts don't match for : VirtualThread-unparker" Reviewed-by: sspitsyn, kevinw
Reviewed-by: dcubed
Reviewed-by: dcubed
…hive location Reviewed-by: mbaesken, erikj
…d_blob(pc())) failed: Must be the same" or SIGSEGV Reviewed-by: coleenp, sspitsyn
Reviewed-by: martin, dholmes
…ng threads Reviewed-by: alanb, dcubed
Reviewed-by: lmesnik
… classes Reviewed-by: mchung
Reviewed-by: jjg
Reviewed-by: vromero
…erifyURL Reviewed-by: weijun
…gine test templates Reviewed-by: rhalade
Reviewed-by: dcubed
Reviewed-by: dfuchs
Reviewed-by: mullan
…en linking downcall handle Reviewed-by: kvn, vlivanov
Added tag jdk-21+11 for changeset fcaf871
Added tag jdk-21+12 for changeset 6e19387
Added tag jdk-21+13 for changeset 02875e7
Added tag jdk-21+14 for changeset 2b81fae
/summary Merge jdk-21+14 |
👋 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 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 8367d28. |
Webrevs |
Merge jdk-21+14
Progress
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/850/head:pull/850
$ git checkout pull/850
Update a local copy of the PR:
$ git checkout pull/850
$ git pull https://git.openjdk.org/valhalla.git pull/850/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 850
View PR using the GUI difftool:
$ git pr show -t 850
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/850.diff