org.objectweb.asm.tree.analysis.Frame

Eric, Remi,

  I wonder if you have spare cycles to look at few things in 
org.objectweb.asm.tree.analysis.Frame class and run some performance 
tests with it.

  First of all it is using a reversed for loop, which in my knowledge is 
not very HotSpot friendly [1] and it is better to use a direct loop 
instead, e.g.

                for (int i = Type.getArgumentTypes(desc).length; i > 0; 
--i) {
                    values.add(0, pop());
                }

  Another thin I noticed is that it is using an ArrayList and inserting 
at the beginning, as in the above code snippet. So, it might be better 
to use LinkedList instead.

  The reversed for loop is also used in few other places, including some 
core API and I wonder if we can squeeze some performance out there too.

  regards,
  Eugene

[1] http://wikis.sun.com/display/HotSpotInternals/PerformanceTechniques

-- 
You receive this message as a subscriber of the [email protected] mailing list.
To unsubscribe: mailto:[email protected]
For general help: mailto:[email protected]?subject=help
OW2 mailing lists service home page: http://www.ow2.org/wws