Re: [rvm-research] A bug in GenCopyMutator

> Hello All,
>
> I think I found a bug in GenCopyMutator. The original  
> collectionPhase() in this class is:
>
> public void collectionPhase(short phaseId, boolean primary) {
>     if (global().traceFullHeap()) {
>       if (phaseId == GenCopy.PREPARE) {
>         super.collectionPhase(phaseId, primary);
>         if (global().gcFullHeap) mature.rebind(GenCopy.toSpace());
>         return;
>       }
>     }
>
>     super.collectionPhase(phaseId, primary);
>   }
>
> I think the line "if (global().gcFullHeap)  
> mature.rebind(GenCopy.toSpace());" should be moved to the phase  
> RELEASE. Because the PREPARE of mutator comes before the PREPARE of  
> GenCopy, at that time the toSpace is not yet toggled, so we will  
> premature into the old space which is wrong.
>
> The original pretenure code made the dacapo benchmarks fail(I've  
> tried antlr, fop, pmd). But they can pass with the modified code.
>
> BTW, I worked with version 2.9.2, and I think the bug also exists in  
> later versions.
>
> cheers,
>
> Dingwen
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables unlimited
> royalty-free distribution of the report engine for externally facing
> server and web deployment.
> http://p.sf.net/sfu/businessobjects_______________________________________________
> Jikesrvm-researchers mailing list
> Jikesrvm-researchers@...
> https://lists.sourceforge.net/lists/listinfo/jikesrvm-researchers