Mercurial > hg > truffle
changeset 3518:0ffcce571d09
Remove a Sysout debug message
| author | Gilles Duboscq <gilles.duboscq@oracle.com> |
|---|---|
| date | Tue, 09 Aug 2011 13:48:46 +0200 |
| parents | dd26b98b9723 |
| children | 16cee060c446 |
| files | ProblemsIdeas.txt graal/com.oracle.max.graal.compiler/src/com/oracle/max/graal/compiler/target/amd64/AMD64LIRAssembler.java |
| diffstat | 2 files changed, 12 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/ProblemsIdeas.txt Mon Aug 08 23:45:59 2011 +0200 +++ b/ProblemsIdeas.txt Tue Aug 09 13:48:46 2011 +0200 @@ -83,3 +83,15 @@ * Hints on register pressure Sometimes we can make better decisions if we know the register pressure, it would be nice to have a way to know about it. Maybe we have register allocation on SSA we can somehow interact with it and try to lower the pressure in some areas on request? + +* Remove <0 check on array allocation when possible + + The XIR templates for array allocation check the length argument agaisnt 0, this can be avoided if the arguent is known to be >= 0 + +* Fuse Div/Rem that act on the same inputs + + Doing it in HIR require a node with 2 outputs which means projections, maybe a special Rem node that takes only the Div as input could do : that would be only one "Projection" node + +* Fuse multiple conditional that use the same BooleanNode condition when doing LIRGen + + Would allow to transform conditional constructs more agressively. Fuse them together or if there is a If that uses the condition, emit them in the if branches
--- a/graal/com.oracle.max.graal.compiler/src/com/oracle/max/graal/compiler/target/amd64/AMD64LIRAssembler.java Mon Aug 08 23:45:59 2011 +0200 +++ b/graal/com.oracle.max.graal.compiler/src/com/oracle/max/graal/compiler/target/amd64/AMD64LIRAssembler.java Tue Aug 09 13:48:46 2011 +0200 @@ -1154,7 +1154,6 @@ Label continuation = new Label(); - System.out.println("gen check" + code); if (GraalOptions.GenSpecialDivChecks && code == LIROpcode.Ldiv) { // check for special case of Long.MIN_VALUE / -1 Label normalCase = new Label();