Reviewers: cromwellian, zundel, jbrosenberg,

Message:
I know this feels a little hacky, but I tracked down most of the sources
of Pruner jitter that's causing it to run more than once.

1) Instance fields needed to be treated like method and put into limbo
until their enclosing type is instantiable.  This shouldn't be too
controversial.

2) A big source of jitter is the fact that CleanupRefsVisitor removes
non-side-effect causing arguments when the invoked parameter is pruned.
Essentially, I am doing the same conceptual thing for arguments to
methods as we are for methods and fields-- unless the argument
expressions have side effects, they are not considered rescued until the
target parameter is rescued.




Please review this at http://gwt-code-reviews.appspot.com/1436802/

Affected files:
  M dev/core/src/com/google/gwt/dev/jjs/impl/ControlFlowAnalyzer.java
  M dev/core/src/com/google/gwt/dev/jjs/impl/Pruner.java


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to