[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-10 Thread Eric Ayers
@Scott If you patch this in and follow the arcane directions, you can get the incremental compile support in blaze. https://mondrian.corp.google.com/changelist/21809612 -Eric. On Thu, Jun 9, 2011 at 5:46 PM, cromwell...@google.com wrote: Overall LGTM. I had to update CFA in my recent CL on

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread jbrosenberg
LGTM w/nitlets http://gwt-code-reviews.appspot.com/1453810/diff/3001/dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java File dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java (right):

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread scottb
Eric, Ray, did you guys want to weigh in also? http://gwt-code-reviews.appspot.com/1453810/diff/3001/dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java File dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java (right):

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread zundel
http://gwt-code-reviews.appspot.com/1453810/diff/3001/dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java File dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java (right):

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread scottb
http://gwt-code-reviews.appspot.com/1453810/diff/3001/dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java File dev/core/src/com/google/gwt/dev/jjs/AstConstructor.java (right):

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread zundel
lgtm 2 http://gwt-code-reviews.appspot.com/1453810/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-09 Thread cromwellian
Overall LGTM. I had to update CFA in my recent CL on class literal optimization to treat an invocation of Object.getClass() as rescuing the class literals of any instantiated types as well as to handle the new Immortal CodeGenTypes. We may have to revisit the UnifyAstVisitor, since it may need

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-08 Thread cromwellian
General question: I'm curious why this couldn't be simpler, that is, why does unification depend on control flow? Couldn't you unify everything without respect to control flow, and then run a pruning pass afterwards reusing the existing CFA code, or is doing it in an integrated pass an attempt to

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-08 Thread Eric Ayers
On Wed, Jun 8, 2011 at 8:40 PM, cromwell...@google.com wrote: General question: I'm curious why this couldn't be simpler, that is, why does unification depend on control flow? Couldn't you unify everything without respect to control flow, and then run a pruning pass afterwards reusing the

[gwt-contrib] Re: JavaAstConstructor uses UnifyAst. (issue1453810)

2011-06-08 Thread Scott Blum
What Eric said. I haven't nailed down all the error reporting yet (although it's closer now), but the idea is that we really can avoid reporting errors on unreachable types. And not reaching code means fewer deserialized types, which is better for memory and takes less time. But there's an even