# HG changeset patch # User Doug Simon # Date 1403182957 -7200 # Node ID 3e3578bc8f9b3e858ab41a3dc2b34747365106bd # Parent 3e40052d3b913b94fa2bfd93cf29771e9879a19a rename: lirLoop* -> loop* diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopEx.java Thu Jun 19 15:02:37 2014 +0200 @@ -39,20 +39,20 @@ public class LoopEx { - private final Loop lirLoop; + private final Loop loop; private LoopFragmentInside inside; private LoopFragmentWhole whole; private CountedLoopInfo counted; // TODO (gd) detect private LoopsData data; private InductionVariables ivs; - LoopEx(Loop lirLoop, LoopsData data) { - this.lirLoop = lirLoop; + LoopEx(Loop loop, LoopsData data) { + this.loop = loop; this.data = data; } - public Loop lirLoop() { - return lirLoop; + public Loop loop() { + return loop; } public LoopFragmentInside inside() { @@ -91,7 +91,7 @@ } public LoopBeginNode loopBegin() { - return (LoopBeginNode) lirLoop().getHeader().getBeginNode(); + return (LoopBeginNode) loop().getHeader().getBeginNode(); } public FixedNode predecessor() { @@ -111,10 +111,10 @@ } public LoopEx parent() { - if (lirLoop.getParent() == null) { + if (loop.getParent() == null) { return null; } - return data.loop(lirLoop.getParent()); + return data.loop(loop.getParent()); } public int size() { @@ -123,7 +123,7 @@ @Override public String toString() { - return (isCounted() ? "CountedLoop [" + counted() + "] " : "Loop ") + "(depth=" + lirLoop().getDepth() + ") " + loopBegin(); + return (isCounted() ? "CountedLoop [" + counted() + "] " : "Loop ") + "(depth=" + loop().getDepth() + ") " + loopBegin(); } private class InvariantPredicate implements NodePredicate { @@ -237,9 +237,9 @@ if (b == untilBlock) { continue; } - if (lirLoop().getExits().contains(b)) { + if (loop().getExits().contains(b)) { exits.add((LoopExitNode) b.getBeginNode()); - } else if (lirLoop().getBlocks().contains(b)) { + } else if (loop().getBlocks().contains(b)) { blocks.add(b.getBeginNode()); work.addAll(b.getDominated()); } diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragment.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragment.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragment.java Thu Jun 19 15:02:37 2014 +0200 @@ -302,7 +302,7 @@ protected void mergeEarlyExits() { assert isDuplicate(); StructuredGraph graph = graph(); - for (BeginNode earlyExit : LoopFragment.toHirBlocks(original().loop().lirLoop().getExits())) { + for (BeginNode earlyExit : LoopFragment.toHirBlocks(original().loop().loop().getExits())) { LoopExitNode loopEarlyExit = (LoopExitNode) earlyExit; FixedNode next = loopEarlyExit.next(); if (loopEarlyExit.isDeleted() || !this.original().contains(loopEarlyExit)) { diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragmentWhole.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragmentWhole.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopFragmentWhole.java Thu Jun 19 15:02:37 2014 +0200 @@ -56,8 +56,8 @@ @Override public NodeBitMap nodes() { if (nodes == null) { - Loop lirLoop = loop().lirLoop(); - nodes = LoopFragment.computeNodes(graph(), LoopFragment.toHirBlocks(lirLoop.getBlocks()), LoopFragment.toHirExits(lirLoop.getExits())); + Loop loop = loop().loop(); + nodes = LoopFragment.computeNodes(graph(), LoopFragment.toHirBlocks(loop.getBlocks()), LoopFragment.toHirExits(loop.getExits())); } return nodes; } diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopsData.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopsData.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/LoopsData.java Thu Jun 19 15:02:37 2014 +0200 @@ -34,7 +34,7 @@ public class LoopsData { - private Map, LoopEx> lirLoopToEx = newIdentityMap(); + private Map, LoopEx> loopToEx = newIdentityMap(); private Map loopBeginToEx = newNodeIdentityMap(); private ControlFlowGraph cfg; @@ -45,15 +45,15 @@ throw Debug.handle(e); } - for (Loop lirLoop : cfg.getLoops()) { - LoopEx ex = new LoopEx(lirLoop, this); - lirLoopToEx.put(lirLoop, ex); + for (Loop loop : cfg.getLoops()) { + LoopEx ex = new LoopEx(loop, this); + loopToEx.put(loop, ex); loopBeginToEx.put(ex.loopBegin(), ex); } } - public LoopEx loop(Loop lirLoop) { - return lirLoopToEx.get(lirLoop); + public LoopEx loop(Loop loop) { + return loopToEx.get(loop); } public LoopEx loop(LoopBeginNode loopBegin) { @@ -61,7 +61,7 @@ } public Collection loops() { - return lirLoopToEx.values(); + return loopToEx.values(); } public List outterFirst() { @@ -70,7 +70,7 @@ @Override public int compare(LoopEx o1, LoopEx o2) { - return o1.lirLoop().getDepth() - o2.lirLoop().getDepth(); + return o1.loop().getDepth() - o2.loop().getDepth(); } }); return loops; @@ -82,7 +82,7 @@ @Override public int compare(LoopEx o1, LoopEx o2) { - return o2.lirLoop().getDepth() - o1.lirLoop().getDepth(); + return o2.loop().getDepth() - o1.loop().getDepth(); } }); return loops; diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopSafepointEliminationPhase.java --- a/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopSafepointEliminationPhase.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.loop/src/com/oracle/graal/loop/phases/LoopSafepointEliminationPhase.java Thu Jun 19 15:02:37 2014 +0200 @@ -37,7 +37,7 @@ if (context.getOptimisticOptimizations().useLoopLimitChecks()) { loops.detectedCountedLoops(); for (LoopEx loop : loops.countedLoops()) { - if (loop.lirLoop().getChildren().isEmpty() && loop.counted().getStamp().getBits() <= 32) { + if (loop.loop().getChildren().isEmpty() && loop.counted().getStamp().getBits() <= 32) { boolean hasSafepoint = false; for (LoopEndNode loopEnd : loop.loopBegin().loopEnds()) { hasSafepoint |= loopEnd.canSafepoint(); @@ -54,7 +54,7 @@ for (LoopEx loop : loops.countedLoops()) { for (LoopEndNode loopEnd : loop.loopBegin().loopEnds()) { Block b = loops.controlFlowGraph().blockFor(loopEnd); - blocks: while (b != loop.lirLoop().getHeader()) { + blocks: while (b != loop.loop().getHeader()) { assert b != null; for (FixedNode node : b.getNodes()) { if (node instanceof Invoke || node instanceof ForeignCallNode) { diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/PartialEvaluationTest.java --- a/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/PartialEvaluationTest.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.truffle.test/src/com/oracle/graal/truffle/test/PartialEvaluationTest.java Thu Jun 19 15:02:37 2014 +0200 @@ -138,7 +138,7 @@ @Override public int compare(LoopEx o1, LoopEx o2) { - return o2.lirLoop().getDepth() - o1.lirLoop().getDepth(); + return o2.loop().getDepth() - o1.loop().getDepth(); } }); return sortedLoops; diff -r 3e40052d3b91 -r 3e3578bc8f9b graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java --- a/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java Thu Jun 19 12:07:02 2014 +0200 +++ b/graal/com.oracle.graal.truffle/src/com/oracle/graal/truffle/PartialEvaluator.java Thu Jun 19 15:02:37 2014 +0200 @@ -297,7 +297,7 @@ @Override public int compare(LoopEx o1, LoopEx o2) { - return o2.lirLoop().getDepth() - o1.lirLoop().getDepth(); + return o2.loop().getDepth() - o1.loop().getDepth(); } }); return sortedLoops;