8384108: [lworld] Update runtime/clone tests with value classes#2416
8384108: [lworld] Update runtime/clone tests with value classes#2416shivshah-oracle wants to merge 4 commits into
Conversation
|
👋 Welcome back shivshah-oracle! A progress list of the required criteria for merging this PR into |
|
❗ This change is not yet ready to be integrated. |
Webrevs
|
lmesnik
left a comment
There was a problem hiding this comment.
Is it possible to combine them with existing tests to reduce code duplication.
|
@shivshah-oracle This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a |
Added CloneWithValueFields test to verify clone behavior with value class fields and arrays
Tests cover field cloning, independence, null handling, inheritance, array cloning, and GC survival using migrated value class types (Integer, Long, Double, Short)
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/valhalla.git pull/2416/head:pull/2416$ git checkout pull/2416Update a local copy of the PR:
$ git checkout pull/2416$ git pull https://git.openjdk.org/valhalla.git pull/2416/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 2416View PR using the GUI difftool:
$ git pr show -t 2416Using diff file
Download this PR as a diff file:
https://git.openjdk.org/valhalla/pull/2416.diff
Using Webrev
Link to Webrev Comment