Review request (XS): 7125516: G1: ~ConcurrentMark() frees incorrectly

Srinivas Ramakrishna ysr1729 at gmail.com
Fri Dec 30 21:31:34 UTC 2011


Looks like that should have taken _tasks.

-- ramki

On Wed, Dec 28, 2011 at 8:04 AM, Stefan Karlsson <stefan.karlsson at oracle.com
> wrote:

> http://cr.openjdk.java.net/~**stefank/7125516/webrev.00/<http://cr.openjdk.java.net/%7Estefank/7125516/webrev.00/>
>
> The ConcurrentMark desctructor _max_task_num which is a size_t and not a
> pointer to allocated memory. However, the ConcurrentMark desctructor is
> never called, so the biggest problem right now is that newer g++ compilers
> complain about this.
>
> Removed the broken, unused ConcurrentMark desctructor code.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/hotspot-gc-dev/attachments/20111230/49b3d88f/attachment.htm>


More information about the hotspot-gc-dev mailing list