RFR (sh/jdk8): Backport 2020-02-27, batch 2

Zhengyu Gu zgu at redhat.com
Thu Feb 27 20:32:14 UTC 2020


Hi Roman,

x86_32 bit does not build.

/home/zgu/ws/sh-jdk8u/hotspot/src/cpu/x86/vm/c1_Runtime1_x86.cpp: In 
static member function ‘static OopMapSet* 
Runtime1::generate_code_for(Runtime1::StubID, StubAssembler*)’:
/home/zgu/ws/sh-jdk8u/hotspot/src/cpu/x86/vm/c1_Runtime1_x86.cpp:1788:28: 
error: ‘c_rarg0’ was not declared in this scope
  1788 |         f.load_argument(0, c_rarg0);
       |                            ^~~~~~~


-Zhengyu

On 2/27/20 3:25 PM, Roman Kennke wrote:
> This is the second batch of backports already (~1 one more expected
> soon). It backports the following issues from sh/jdk11:
> 
> JDK-8224525: Shenandoah: Eliminate shenandoah verifier's side-effects
> JDK-8225171: Remove leftovers in shenandoahBarrierSetC1.cpp
> JDK-8226586: Shenandoah: No need to pre-evacuate roots for degenerated GC
> JDK-8227676: Shenandoah: More consistent naming of LRB runtime entry points
> JDK-8228364: Shenandoah: Remove unused code from ShenandoahBarrierSetC1
> JDK-8228369: Shenandoah: Refactor LRB C1 stubs
> JDK-8229470: Shenandoah: Fix C1 getAndSetObject() failure
> JDK-8229709: x86_32 build and test failures after JDK-8228369
> (Shenandoah: Refactor LRB C1 stubs)
> JDK-8229865: Use explicit #include debug.hpp for STATIC_ASSERT in
> gc/shenandoah/shenandoahUtils.cpp
> 
> The significant change (and diverging from sh/jdk11) is JDK-8228369, it
> requires some addition to c1_Runtime_$ARCH.cpp that is
> difficult/impossible to put into shenandoahBarrierSetAssembler_$ARCH.cpp
> because of missing GC interfaces. But it's overall contained, so should
> be ok.
> 
> "JDK-8229470: Shenandoah: Fix C1 getAndSetObject() failure" is
> interesting because it's wholly different from jdk11 because of the
> recent (2 days ago) getAndSetObject() fix: instead of
> overriding/reverting that I keep the recent fix and sneak in the aarch64
> version of it (which we've forgotten about 2 days ago) :-) I hope that's
> ok. It's in the spirit of the original fix I suppose.
> 
> Changesets:
> http://cr.openjdk.java.net/~rkennke/backport-shjdk8-2020-02-27-2/changesets.txt
> Webrev:
> http://cr.openjdk.java.net/~rkennke/backport-shjdk8-2020-02-27-2/webrev.00/
> 
> Testing: hotspot_gc_shenandoah (x86_64, aarch64): somebody should
> probably check x86_32 bit because I currently can't. Aarch64 shows some
> failures but no regressions: I'll check those once I'm through with all
> backports. They've been pre-existing before I started with the backports
> queue.
> 
> Good?
> 
> Thank you,
> Roman
> 



More information about the shenandoah-dev mailing list