Request for Reviews(M): 7092905: C2: Keep track of the number of dead nodes (original) (raw)
Bharadwaj Yadavalli bharadwaj.yadavalli at oracle.com
Fri Nov 2 11:16:11 PDT 2012
- Previous message: Request for Reviews(M): 7092905: C2: Keep track of the number of dead nodes
- Next message: Code review request: NMT: assertion failed: assert(rec->addr() + rec->size() <= cur->base()) failed: Can not overlap in memSnapshot.cpp
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 11/2/2012 12:44 PM, Roland Westrelin wrote:
Hi Bharadwaj,
I'd like to get a code review of the changes at http://cr.openjdk.java.net/~bharadwaj/7092905/webrev01/ Shouldn't Compile::deadnodecount be set to deadnodelist.size() after/in a call to Compile::identifyusefulnodes()?
Yes. That is correct. I have fixed that.
Thanks for reviewing the code. I plan on sending out soon a new webrev incorporating all the review comments I received.
Thanks,
Bharadwaj
- Previous message: Request for Reviews(M): 7092905: C2: Keep track of the number of dead nodes
- Next message: Code review request: NMT: assertion failed: assert(rec->addr() + rec->size() <= cur->base()) failed: Can not overlap in memSnapshot.cpp
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]