RFR(XS) 8223215: Shenandoah: Support verifying subset of roots

Aleksey Shipilev shade at redhat.com
Mon May 13 10:39:43 UTC 2019


On 5/10/19 11:14 PM, Zhengyu Gu wrote:
> 
> On 5/10/19 3:12 PM, Roman Kennke wrote:
>> Hi Zhengyu,
>>
>> this looks good.
>> Maybe the root-verifier code should be in its own files?
> 
> Updated: http://cr.openjdk.java.net/~zgu/JDK-8223215/webrev.01/

Stylistic nits:

*) New line is missing here in shenandoahRootVerifier.cpp?

  54   }
  55   if (verify(CLDGRoots)) {

*) You sure uint is not too narrow here? I think it's fine, but maybe casting through uintptr_t is
safer?

  42 void ShenandoahRootVerifier::excludes(RootTypes types) {
  43   _types = static_cast<ShenandoahRootVerifier::RootTypes>(static_cast<uint>(_types) &
(~static_cast<uint>(types)));
  44 }


-Aleksey



More information about the shenandoah-dev mailing list