hg: graal/graal: 24 new changesets

doug.simon at oracle.com doug.simon at oracle.com
Sat May 17 01:00:06 UTC 2014 

Changeset: eaeba148bb15
Author:    Tom Rodriguez <tom.rodriguez at oracle.com>
Date:      2014-05-15 20:11 -0700
URL:       http://hg.openjdk.java.net/graal/graal/rev/eaeba148bb15

more aggressively fold implicit nulls into memory operations

! graal/com.oracle.graal.nodes/src/com/oracle/graal/nodes/extended/NullCheckNode.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/UseTrappingNullChecksPhase.java

Changeset: 423bf61c9c32
Author:    Tom Rodriguez <tom.rodriguez at oracle.com>
Date:      2014-05-16 00:12 -0700
URL:       http://hg.openjdk.java.net/graal/graal/rev/423bf61c9c32

use inner classes instead of reflection during matching

! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchGenerator.java
! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchProcessor.java
! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchStatement.java

Changeset: 98423229008c
Author:    Tom Rodriguez <tom.rodriguez at oracle.com>
Date:      2014-05-16 00:12 -0700
URL:       http://hg.openjdk.java.net/graal/graal/rev/98423229008c

allow overriding the NodeClass lookup when building MatchStatements

! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchPattern.java
! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchProcessor.java
! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchRuleRegistry.java
! graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/match/MatchStatementSet.java

Changeset: 16059f6f5661
Author:    Doug Simon <doug.simon at oracle.com>
Date:      2014-05-16 12:06 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/16059f6f5661

mx: drain *all* output from subprocess if redirecting to functions

! mxtool/mx.py

Changeset: c66c28e1f866
Author:    Doug Simon <doug.simon at oracle.com>
Date:      2014-05-16 12:23 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/c66c28e1f866

minor spelling and modifier fix

! graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/CompilationTask.java
! graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/bridge/VMToCompilerImpl.java

Changeset: bd9be86ce634
Author:    Lukas Stadler <lukas.stadler at oracle.com>
Date:      2014-05-16 14:01 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/bd9be86ce634

do not reprofile upon exceptions thrown in NewArrayStub or NewInstanceStub

! graal/com.oracle.graal.hotspot/src/com/oracle/graal/hotspot/stubs/StubUtil.java

Changeset: c5ff0e1e53cf
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 11:59 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/c5ff0e1e53cf

[inlining] fixing input as instance final rather than passing it over and over

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningPhase.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/InliningData.java

Changeset: 619b194bb2a3
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 12:03 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/619b194bb2a3

[inlining] consumer becomes initializer of the probabilities map

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningPhase.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/InliningData.java

Changeset: 8e2ed3a0812a
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 13:39 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/8e2ed3a0812a

[inlining] typos in source comment

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningPhase.java

Changeset: eee32e28151a
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 14:44 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/eee32e28151a

[inlining] reduced verbosity in checkTargetConditions()

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: e5876a0f2453
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 14:20 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/e5876a0f2453

[inlining] replaced method body with call to code duplicate

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: b67a0a440ecb
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 14:47 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/b67a0a440ecb

[inlining] one less logging method to worry about

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: 8344485d71bd
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 14:57 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/8344485d71bd

[inlining] pulling side-effects (logging) out of method that evals a condition

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: 1e3be4d992f3
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:14 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/1e3be4d992f3

[inlining] tradeoff: "return null" still shorter than "return ...AndReturnNull"

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: 1d26d24a572e
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:24 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/1d26d24a572e

[inlining] "return null" favored again over "return ...AndReturnNull"

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: b68907673c8e
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:28 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/b68907673c8e

[inlining] shorter and equally informative, logNotInlined vs logNotInlinedMethod

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: da81af70a3ef
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:30 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/da81af70a3ef

[inlining] another case of logNotInlined vs logNotInlinedMethod

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: bf6db624bf62
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:46 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/bf6db624bf62

[inlining] no need for the suspense about return value

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/GreedyInliningPolicy.java

Changeset: 9e697dbc28a7
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 15:54 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/9e697dbc28a7

[inlining] redux, no need for the suspense about return value

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/GreedyInliningPolicy.java

Changeset: 13fa2732a8bd
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 16:00 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/13fa2732a8bd

[inlining] readability by means of import static

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/GreedyInliningPolicy.java

Changeset: 7af25e98aa36
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 16:02 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/7af25e98aa36

[inlining] untangling concerns, micro-step by micro-step

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: a508335e5ee8
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 16:29 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/a508335e5ee8

[inlining] no need for guessing a return value that doesn't matter

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java

Changeset: 0b363d4ceb84
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 16:36 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/0b363d4ceb84

[inlining] behavior becomes less argument-dependent, arguments become redundant

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/GreedyInliningPolicy.java

Changeset: 1e3f19292a32
Author:    Miguel Garcia <miguel.m.garcia at oracle.com>
Date:      2014-05-16 19:59 +0200
URL:       http://hg.openjdk.java.net/graal/graal/rev/1e3f19292a32

[inlining] reverting refactoring trail until spoiling commit(s) are discovered

! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningPhase.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/InliningUtil.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/policy/GreedyInliningPolicy.java
! graal/com.oracle.graal.phases.common/src/com/oracle/graal/phases/common/inlining/walker/InliningData.java

More information about the graal-dev mailing list