<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> > If the old serial version (old code) is kept, this condition needs to be<br>
> > adapted, as data structures within the if-block started at that line are<br>
> > only used in the parallel case.<br>
><br>
> Where talking about this like<br>
><br>
>    if ((CMSParallelRemarkEnabled && CMSParallelSurvivorRemarkEnabled) ||<br>
> CMSParallelInitialMarkEnabled) {<br>
><br>
> and you're saying the<br>
> > this condition needs to be<br>
> > adapted, as data structures within the if-block started at that line are<br>
> > only used in the parallel case.<br>
> Sorry, I still don't get it.  Are you saying the condition in the test is<br>
> not correct?  By "parallel case" do you mean the case where<br>
> more than one GC thread does the work?  Or are you counting<br>
> one GC thread executing the parallel code as part of the<br>
> "parallel case"?<br>
<br>
</div>Sorry for confusing you. This was just some suggestion for if one of the<br>
serial paths is removed (e.g. in the sense of if we remove that, we need<br>
to also remove this). Since it has been decided to keep both for now (I<br>
think), this comment is obsolete.</blockquote><div><br></div><div style>I take this point as a non-issue. </div></div></div></div>