Skip to content
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 #511

Closed
wants to merge 107 commits into from
Closed

Merge jdk #511

wants to merge 107 commits into from

Conversation

MrSimms
Copy link
Member

@MrSimms MrSimms commented Jul 30, 2021

Merge tag 'jdk-18+8'


Progress

  • Change must not contain extraneous whitespace

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/valhalla pull/511/head:pull/511
$ git checkout pull/511

Update a local copy of the PR:
$ git checkout pull/511
$ git pull https://git.openjdk.java.net/valhalla pull/511/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 511

View PR using the GUI difftool:
$ git pr show -t 511

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/valhalla/pull/511.diff

Daniel D. Daugherty and others added 30 commits Jul 21, 2021
…App/MultiWindowAppTest.java

Reviewed-by: prr
Reviewed-by: jlahoda, rhalade, mschoene
Reviewed-by: rhalade, psadhukhan, mschoene, serb
Reviewed-by: rriggs, rhalade
Reviewed-by: weijun, rhalade
Reviewed-by: hchao, valeriep
Reviewed-by: rhalade, prr, vdyakov, ahgross
Reviewed-by: rhalade, mschoene, psadhukhan, jdv, serb
Reviewed-by: dfuchs, michaelm, rhalade, ahgross
Reviewed-by: ahgross, rhalade, aefimov
Reviewed-by: lancea, naoto, ahgross, rhalade
Reviewed-by: vlivanov, ahgross, thartmann, rhalade
Reviewed-by: michaelm, aefimov, dfuchs, chegar, ahgross, rhalade
Reviewed-by: ahgross, kvn, rhalade, thartmann
…pendencies::is_concrete_method(m, ctxk)) failed: mismatch

Reviewed-by: vlivanov, thartmann
Reviewed-by: simonis, dholmes
Co-authored-by: Robbin Ehn <rehn@openjdk.org>
Co-authored-by: Patricio Chilano Mateo <pchilanomate@openjdk.org>
Reviewed-by: dcubed, dholmes, coleenp
Reviewed-by: kbarrett, iwalulya
Bradford Wetmore and others added 15 commits Jul 27, 2021
…ead of shouldContaint

Reviewed-by: mseledtsov, dholmes
…X:TieredStopAtLevel=1

Reviewed-by: cjplummer, kvn
Reviewed-by: psandoz, sviswanathan
…d text color

Reviewed-by: psadhukhan, pbansal
…lper methods in the interpreter

Reviewed-by: shade
@MrSimms
Copy link
Member Author

MrSimms commented Jul 30, 2021

/summary Merge tag 'jdk-18+8'

@bridgekeeper
Copy link

bridgekeeper bot commented Jul 30, 2021

👋 Welcome back dsimms! A progress list of the required criteria for merging this PR into lworld will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link

openjdk bot commented Jul 30, 2021

@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:

Merge jdk

Merge tag 'jdk-18+8'

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 1 new commit pushed to the lworld branch:

  • 6699537: 8271531: [lworld] Implicit null check optimization does not hoist constant load input

Please see this link for an up-to-date comparison between the source branch of this pull request and the lworld branch.
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 lworld branch, type /integrate in a new comment.

@openjdk
Copy link

openjdk bot commented Jul 30, 2021

@MrSimms Setting summary to Merge tag 'jdk-18+8'

@mlbridge
Copy link

mlbridge bot commented Jul 30, 2021

Webrevs

@MrSimms
Copy link
Member Author

MrSimms commented Jul 30, 2021

/integrate

@openjdk
Copy link

openjdk bot commented Jul 30, 2021

Going to push as commit 2ca8eba.
Since your change was applied there have been 2 commits pushed to the lworld branch:

  • c8b88f2: 8271535: [lworld] TestLWorld fails during IR verification because method is not inlined
  • 6699537: 8271531: [lworld] Implicit null check optimization does not hoist constant load input

Your commit was automatically rebased without conflicts.

@openjdk openjdk bot closed this Jul 30, 2021
@openjdk openjdk bot added integrated and removed ready rfr labels Jul 30, 2021
@openjdk
Copy link

openjdk bot commented Jul 30, 2021

@MrSimms Pushed as commit 2ca8eba.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.

@MrSimms MrSimms deleted the lworld_merge_jdk_18_8 branch Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment