RFR 8225014: Separate ShenandoahRootScanner method for object_iterate

[Zhengyu Gu]

On 5/30/19 5:16 AM, Aleksey Shipilev wrote:
> On 5/30/19 2:52 AM, Zhengyu Gu wrote:
>> Please review this patch that allows SH::object_iterate() invokes ShenandoahRootScanner method
>> without assertion. A regression test is also added.
>>
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8225014
>> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8225014/webrev.00/
> 
> *) Suggestion for the name "roots_do_no_assert" -> "roots_do_unchecked"
> 
> *) I was thinking that object_iterate should visit the code cache too. Current shared method would
> do that optionally depending on ShenandoahConcurrentScanCodeRoots. Maybe it is cleaner to just do
> the separate "roots_do_all" specifically for heap iteration?
Right.

Updated: http://cr.openjdk.java.net/~zgu/JDK-8225014/webrev.01/index.html

Thanks,

-Zhengyu

> 
> Regression test looks nice.
>