-
Notifications
You must be signed in to change notification settings - Fork 125
draft lworld #1669
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?
draft lworld #1669
Conversation
… not applied Reviewed-by: rriggs
Co-authored-by: Vicente Romero <[email protected]> Reviewed-by: coleenp, fparain, liach
…diately upon entry to a non-this-calling constructor
…07.java to don't faile with compilation error Reviewed-by: rriggs
…an_see_stored_value Reviewed-by: chagedorn
…ict field assignment Reviewed-by: vromero
…XX:+UseNullableValueFlattening
…eld has not been initialized before the super invocation
Reviewed-by: coleenp
… VarHandle Reviewed-by: mcimadamore
Reviewed-by: liach
…8411 fails Reviewed-by: thartmann
… just sets a strict field
Reviewed-by: thartmann
…AsFields flag from tests Reviewed-by: thartmann
…izers Reviewed-by: mcimadamore
…field_offset Reviewed-by: thartmann
…with -XX:+UseAtomicValueFlattening
…e annotation Co-authored-by: Tobias Hartmann <[email protected]> Co-authored-by: Roger Riggs <[email protected]>
Reviewed-by: jbhateja, thartmann
…chk == 0) failed: Field too big for insn Reviewed-by: mhaessig
…etLocal.java fails with JTREG_TEST_THREAD_FACTORY=Virtual
… strict instance fields Co-authored-by: Dan Heidinga <[email protected]> Reviewed-by: heidinga
Reviewed-by: thartmann
…a.lang.Object::<init> Reviewed-by: thartmann
…hat would not be allowed in the prologue Reviewed-by: liach
Reviewed-by: thartmann
…lure Reviewed-by: fparain
…ssignment javac tests
… this() invocation Reviewed-by: liach
Reviewed-by: thartmann
…rict instance fields Reviewed-by: chagedorn
Reviewed-by: thartmann
…rtedClassVersionError Reviewed-by: dsimms, rriggs
👋 Welcome back rhalade! A progress list of the required criteria for merging this PR into |
❗ This change is not yet ready to be integrated. |
@rhalade this pull request can not be integrated into git checkout lworld
git fetch https://git.openjdk.org/valhalla.git master
git merge FETCH_HEAD
# resolve conflicts and follow the instructions given by git merge
git commit -m "Merge master"
git push |
To avoid this situation, create a new branch for your changes and reset the
Then proceed to create a new pull request with |
|
…elds with initializers in the prologue Reviewed-by: liach
…truction.java Reviewed-by: thartmann
Progress
Error
Integration blockers
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/1669/head:pull/1669
$ git checkout pull/1669
Update a local copy of the PR:
$ git checkout pull/1669
$ git pull https://git.openjdk.org/valhalla.git pull/1669/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 1669
View PR using the GUI difftool:
$ git pr show -t 1669
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/1669.diff