On 08/01/2017 05:49 PM, Zhengyu Gu wrote: > Okay, updated: > > Webrev: http://cr.openjdk.java.net/~zgu/shenandoah/sh_printer/webrev.00/ Looks good. But pretty sure you haven't updated recently, because the closure is now called ShenandoahPrintAllRefsOopClosure. -Aleksey