http://webrev.invokedynamic.info/kvn/6670459/index.html

Fixed 6670459: Fix Node::dump() performance

Problem:
The fix for 6614330 introduced a performance problem in Node::dump() -
a dump of whole Ideal graph takes forever.

Solution:
I replaced depth-first iteration with breadth-first.
The code is very simple and fast. But I had to remove
Ross's search for loops (which don't have phis and regions).

I also fxied the old bug: Node_Stack.size() incorrectly
returns big number instead of 0 when it is empty.

Reviewed by:
Fix verified (y/n): y, failed test case.

Other testing:
PRT

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 1 of 1 | next ›
Discussion Overview
grouphotspot-compiler-dev @
categoriesopenjdk
postedMar 3, '08 at 9:33a
activeMar 3, '08 at 9:33a
posts1
users1
websiteopenjdk.java.net

1 user in discussion

Vladimir Kozlov: 1 post

People

Translate

site design / logo © 2021 Grokbase