src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.phases.common/src/org/graalvm/compiler/phases/common/FixReadsPhase.java
author chegar
Thu, 17 Oct 2019 20:54:25 +0100
branchdatagramsocketimpl-branch
changeset 58679 9c3209ff7550
parent 58678 9cf78a70fa4f
parent 58299 6df94ce3ab2f
permissions -rw-r--r--
datagramsocketimpl-branch: merge with default
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     1
/*
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 55509
diff changeset
     2
 * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     4
 *
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     7
 * published by the Free Software Foundation.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     8
 *
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    13
 * accompanied this code).
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    14
 *
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    18
 *
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    21
 * questions.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    22
 */
50858
2d3e99a72541 8205824: Update Graal
never
parents: 49873
diff changeset
    23
2d3e99a72541 8205824: Update Graal
never
parents: 49873
diff changeset
    24
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    25
package org.graalvm.compiler.phases.common;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    26
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48861
diff changeset
    27
import jdk.internal.vm.compiler.collections.EconomicMap;
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 55509
diff changeset
    28
import jdk.internal.vm.compiler.collections.Equivalence;
49873
26ebfe8ce852 8199755: Update Graal
dlong
parents: 48861
diff changeset
    29
import jdk.internal.vm.compiler.collections.MapCursor;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    30
import org.graalvm.compiler.core.common.GraalOptions;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    31
import org.graalvm.compiler.core.common.cfg.BlockMap;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    32
import org.graalvm.compiler.core.common.spi.ConstantFieldProvider;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    33
import org.graalvm.compiler.core.common.type.FloatStamp;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    34
import org.graalvm.compiler.core.common.type.Stamp;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    35
import org.graalvm.compiler.core.common.type.StampFactory;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    36
import org.graalvm.compiler.debug.CounterKey;
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    37
import org.graalvm.compiler.debug.DebugContext;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    38
import org.graalvm.compiler.graph.Node;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    39
import org.graalvm.compiler.graph.NodeMap;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    40
import org.graalvm.compiler.graph.NodeStack;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    41
import org.graalvm.compiler.graph.Position;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    42
import org.graalvm.compiler.graph.spi.CanonicalizerTool;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    43
import org.graalvm.compiler.nodeinfo.InputType;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    44
import org.graalvm.compiler.nodes.AbstractBeginNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.AbstractMergeNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    46
import org.graalvm.compiler.nodes.BinaryOpLogicNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    47
import org.graalvm.compiler.nodes.ConstantNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    48
import org.graalvm.compiler.nodes.EndNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    49
import org.graalvm.compiler.nodes.IfNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    50
import org.graalvm.compiler.nodes.LogicNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    51
import org.graalvm.compiler.nodes.MergeNode;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
    52
import org.graalvm.compiler.nodes.NodeView;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    53
import org.graalvm.compiler.nodes.PhiNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    54
import org.graalvm.compiler.nodes.PiNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    55
import org.graalvm.compiler.nodes.StructuredGraph;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    56
import org.graalvm.compiler.nodes.StructuredGraph.ScheduleResult;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    57
import org.graalvm.compiler.nodes.UnaryOpLogicNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    58
import org.graalvm.compiler.nodes.ValueNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    59
import org.graalvm.compiler.nodes.ValuePhiNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    60
import org.graalvm.compiler.nodes.calc.BinaryNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    61
import org.graalvm.compiler.nodes.calc.ConditionalNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    62
import org.graalvm.compiler.nodes.calc.UnaryNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    63
import org.graalvm.compiler.nodes.cfg.Block;
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
    64
import org.graalvm.compiler.nodes.cfg.ControlFlowGraph;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    65
import org.graalvm.compiler.nodes.cfg.ControlFlowGraph.RecursiveVisitor;
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
    66
import org.graalvm.compiler.nodes.extended.GuardingNode;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    67
import org.graalvm.compiler.nodes.extended.IntegerSwitchNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    68
import org.graalvm.compiler.nodes.memory.FixedAccessNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    69
import org.graalvm.compiler.nodes.memory.FloatingAccessNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    70
import org.graalvm.compiler.nodes.memory.FloatingReadNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    71
import org.graalvm.compiler.nodes.memory.MemoryAccess;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    72
import org.graalvm.compiler.nodes.memory.MemoryPhiNode;
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents: 54084
diff changeset
    73
import org.graalvm.compiler.nodes.spi.CoreProviders;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    74
import org.graalvm.compiler.nodes.util.GraphUtil;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    75
import org.graalvm.compiler.options.OptionValues;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    76
import org.graalvm.compiler.phases.BasePhase;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    77
import org.graalvm.compiler.phases.Phase;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    78
import org.graalvm.compiler.phases.graph.ScheduledNodeIterator;
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
    79
import org.graalvm.compiler.phases.schedule.SchedulePhase;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
    80
import org.graalvm.compiler.phases.schedule.SchedulePhase.SchedulingStrategy;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    81
import org.graalvm.compiler.phases.tiers.LowTierContext;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    82
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    83
import jdk.vm.ci.meta.Assumptions;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    84
import jdk.vm.ci.meta.Constant;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
    85
import jdk.vm.ci.meta.ConstantReflectionProvider;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    86
import jdk.vm.ci.meta.MetaAccessProvider;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    87
import jdk.vm.ci.meta.TriState;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    88
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    89
/**
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    90
 * This phase lowers {@link FloatingReadNode FloatingReadNodes} into corresponding fixed reads.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    91
 */
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    92
public class FixReadsPhase extends BasePhase<LowTierContext> {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
    93
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    94
    private static final CounterKey counterStampsRegistered = DebugContext.counter("FixReads_StampsRegistered");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    95
    private static final CounterKey counterIfsKilled = DebugContext.counter("FixReads_KilledIfs");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    96
    private static final CounterKey counterConditionalsKilled = DebugContext.counter("FixReads_KilledConditionals");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    97
    private static final CounterKey counterCanonicalizedSwitches = DebugContext.counter("FixReads_CanonicalizedSwitches");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    98
    private static final CounterKey counterConstantReplacements = DebugContext.counter("FixReads_ConstantReplacement");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    99
    private static final CounterKey counterConstantInputReplacements = DebugContext.counter("FixReads_ConstantInputReplacement");
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   100
    private static final CounterKey counterBetterMergedStamps = DebugContext.counter("FixReads_BetterMergedStamp");
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   101
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   102
    protected boolean replaceInputsWithConstants;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   103
    protected Phase schedulePhase;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   104
46393
d497d892ab11 8178864: Update Graal
iveresov
parents: 46344
diff changeset
   105
    @Override
d497d892ab11 8178864: Update Graal
iveresov
parents: 46344
diff changeset
   106
    public float codeSizeIncrease() {
d497d892ab11 8178864: Update Graal
iveresov
parents: 46344
diff changeset
   107
        return 2.0f;
d497d892ab11 8178864: Update Graal
iveresov
parents: 46344
diff changeset
   108
    }
d497d892ab11 8178864: Update Graal
iveresov
parents: 46344
diff changeset
   109
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   110
    private static class FixReadsClosure extends ScheduledNodeIterator {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   111
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   112
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   113
        protected void processNode(Node node) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   114
            if (node instanceof AbstractMergeNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   115
                AbstractMergeNode mergeNode = (AbstractMergeNode) node;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   116
                for (MemoryPhiNode memoryPhi : mergeNode.memoryPhis().snapshot()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   117
                    // Memory phi nodes are no longer necessary at this point.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   118
                    memoryPhi.replaceAtUsages(null);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   119
                    memoryPhi.safeDelete();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   120
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   121
            } else if (node instanceof FloatingAccessNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   122
                FloatingAccessNode floatingAccessNode = (FloatingAccessNode) node;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   123
                floatingAccessNode.setLastLocationAccess(null);
52578
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   124
                GuardingNode guard = floatingAccessNode.getGuard();
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   125
                if (guard != null) {
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   126
                    floatingAccessNode.setGuard(null);
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   127
                    GraphUtil.tryKillUnused(guard.asNode());
7dd81e82d083 8210777: Update Graal
dlong
parents: 50858
diff changeset
   128
                }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   129
                FixedAccessNode fixedAccess = floatingAccessNode.asFixedNode();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   130
                replaceCurrent(fixedAccess);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   131
            } else if (node instanceof PiNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   132
                PiNode piNode = (PiNode) node;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   133
                if (piNode.stamp(NodeView.DEFAULT).isCompatible(piNode.getOriginalNode().stamp(NodeView.DEFAULT))) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   134
                    // Pi nodes are no longer necessary at this point.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   135
                    piNode.replaceAndDelete(piNode.getOriginalNode());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   136
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   137
            } else if (node instanceof MemoryAccess) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   138
                MemoryAccess memoryAccess = (MemoryAccess) node;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   139
                memoryAccess.setLastLocationAccess(null);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   140
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   141
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   142
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   143
    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   144
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   145
    public static class RawConditionalEliminationVisitor implements RecursiveVisitor<Integer> {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   146
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   147
        protected final NodeMap<StampElement> stampMap;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   148
        protected final NodeStack undoOperations;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   149
        private final ScheduleResult schedule;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   150
        private final StructuredGraph graph;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   151
        private final MetaAccessProvider metaAccess;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   152
        private final boolean replaceConstantInputs;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   153
        private final BlockMap<Integer> blockActionStart;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   154
        private final EconomicMap<MergeNode, EconomicMap<ValueNode, Stamp>> endMaps;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   155
        private final DebugContext debug;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   156
        private final RawCanonicalizerTool rawCanonicalizerTool = new RawCanonicalizerTool();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   157
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   158
        private class RawCanonicalizerTool implements NodeView, CanonicalizerTool {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   159
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   160
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   161
            public Assumptions getAssumptions() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   162
                return graph.getAssumptions();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   163
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   164
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   165
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   166
            public MetaAccessProvider getMetaAccess() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   167
                return metaAccess;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   168
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   169
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   170
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   171
            public ConstantReflectionProvider getConstantReflection() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   172
                return null;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   173
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   174
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   175
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   176
            public ConstantFieldProvider getConstantFieldProvider() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   177
                return null;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   178
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   179
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   180
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   181
            public boolean canonicalizeReads() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   182
                return false;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   183
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   184
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   185
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   186
            public boolean allUsagesAvailable() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   187
                return true;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   188
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   189
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   190
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   191
            public Integer smallestCompareWidth() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   192
                return null;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   193
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   194
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   195
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   196
            public OptionValues getOptions() {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   197
                return graph.getOptions();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   198
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   199
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   200
            @Override
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   201
            public Stamp stamp(ValueNode node) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   202
                return getBestStamp(node);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   203
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   204
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   205
        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   206
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   207
        public RawConditionalEliminationVisitor(StructuredGraph graph, ScheduleResult schedule, MetaAccessProvider metaAccess, boolean replaceInputsWithConstants) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   208
            this.graph = graph;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   209
            this.debug = graph.getDebug();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   210
            this.schedule = schedule;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   211
            this.metaAccess = metaAccess;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   212
            blockActionStart = new BlockMap<>(schedule.getCFG());
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 55509
diff changeset
   213
            endMaps = EconomicMap.create(Equivalence.IDENTITY);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   214
            stampMap = graph.createNodeMap();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   215
            undoOperations = new NodeStack();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   216
            replaceConstantInputs = replaceInputsWithConstants && GraalOptions.ReplaceInputsWithConstantsBasedOnStamps.getValue(graph.getOptions());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   217
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   218
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   219
        protected void replaceInput(Position p, Node oldInput, Node newConstantInput) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   220
            p.set(oldInput, newConstantInput);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   221
        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   222
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   223
        protected int replaceConstantInputs(Node node) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   224
            int replacements = 0;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   225
            // Check if we can replace any of the inputs with a constant.
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   226
            for (Position p : node.inputPositions()) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   227
                Node input = p.get(node);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   228
                if (p.getInputType() == InputType.Value) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   229
                    if (input instanceof ValueNode) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   230
                        ValueNode valueNode = (ValueNode) input;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   231
                        if (valueNode instanceof ConstantNode) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   232
                            // Input already is a constant.
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   233
                        } else {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   234
                            Stamp bestStamp = getBestStamp(valueNode);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   235
                            Constant constant = bestStamp.asConstant();
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   236
                            if (constant != null) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   237
                                if (bestStamp instanceof FloatStamp) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   238
                                    FloatStamp floatStamp = (FloatStamp) bestStamp;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   239
                                    if (floatStamp.contains(0.0d)) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   240
                                        // Could also be -0.0d.
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   241
                                        continue;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   242
                                    }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   243
                                }
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   244
                                counterConstantInputReplacements.increment(node.getDebug());
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   245
                                ConstantNode stampConstant = ConstantNode.forConstant(bestStamp, constant, metaAccess, graph);
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   246
                                assert stampConstant.stamp(NodeView.DEFAULT).isCompatible(valueNode.stamp(NodeView.DEFAULT));
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   247
                                replaceInput(p, node, stampConstant);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   248
                                replacements++;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   249
                            }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   250
                        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   251
                    }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   252
                }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   253
            }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   254
            return replacements;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   255
        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   256
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   257
        protected void processNode(Node node) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   258
            assert node.isAlive();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   259
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   260
            if (replaceConstantInputs) {
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   261
                replaceConstantInputs(node);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   262
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   263
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   264
            if (node instanceof MergeNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   265
                registerCombinedStamps((MergeNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   266
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   267
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   268
            if (node instanceof AbstractBeginNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   269
                processAbstractBegin((AbstractBeginNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   270
            } else if (node instanceof IfNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   271
                processIf((IfNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   272
            } else if (node instanceof IntegerSwitchNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   273
                processIntegerSwitch((IntegerSwitchNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   274
            } else if (node instanceof BinaryNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   275
                processBinary((BinaryNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   276
            } else if (node instanceof ConditionalNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   277
                processConditional((ConditionalNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   278
            } else if (node instanceof UnaryNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   279
                processUnary((UnaryNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   280
            } else if (node instanceof EndNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   281
                processEnd((EndNode) node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   282
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   283
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   284
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   285
        protected void registerCombinedStamps(MergeNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   286
            EconomicMap<ValueNode, Stamp> endMap = endMaps.get(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   287
            MapCursor<ValueNode, Stamp> entries = endMap.getEntries();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   288
            while (entries.advance()) {
47798
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   289
                ValueNode value = entries.getKey();
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   290
                if (value.isDeleted()) {
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   291
                    // nodes from this map can be deleted when a loop dies
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   292
                    continue;
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   293
                }
9fe9292f5931 8190710: Update Graal
dlong
parents: 47216
diff changeset
   294
                if (registerNewValueStamp(value, entries.getValue())) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   295
                    counterBetterMergedStamps.increment(debug);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   296
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   297
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   298
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   299
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   300
        protected void processEnd(EndNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   301
            AbstractMergeNode abstractMerge = node.merge();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   302
            if (abstractMerge instanceof MergeNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   303
                MergeNode merge = (MergeNode) abstractMerge;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   304
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   305
                NodeMap<Block> blockToNodeMap = this.schedule.getNodeToBlockMap();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   306
                Block mergeBlock = blockToNodeMap.get(merge);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   307
                Block mergeBlockDominator = mergeBlock.getDominator();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   308
                Block currentBlock = blockToNodeMap.get(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   309
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   310
                EconomicMap<ValueNode, Stamp> currentEndMap = endMaps.get(merge);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   311
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   312
                if (currentEndMap == null || !currentEndMap.isEmpty()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   313
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 55509
diff changeset
   314
                    EconomicMap<ValueNode, Stamp> endMap = EconomicMap.create(Equivalence.IDENTITY);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   315
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   316
                    // Process phis
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   317
                    for (ValuePhiNode phi : merge.valuePhis()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   318
                        if (currentEndMap == null || currentEndMap.containsKey(phi)) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   319
                            ValueNode valueAt = phi.valueAt(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   320
                            Stamp bestStamp = getBestStamp(valueAt);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   321
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   322
                            if (currentEndMap != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   323
                                bestStamp = bestStamp.meet(currentEndMap.get(phi));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   324
                            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   325
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   326
                            if (!bestStamp.equals(phi.stamp(NodeView.DEFAULT))) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   327
                                endMap.put(phi, bestStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   328
                            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   329
                        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   330
                    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   331
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   332
                    int lastMark = undoOperations.size();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   333
                    while (currentBlock != mergeBlockDominator) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   334
                        int mark = blockActionStart.get(currentBlock);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   335
                        for (int i = lastMark - 1; i >= mark; --i) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   336
                            ValueNode nodeWithNewStamp = (ValueNode) undoOperations.get(i);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   337
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   338
                            if (nodeWithNewStamp.isDeleted() || nodeWithNewStamp instanceof LogicNode || nodeWithNewStamp instanceof ConstantNode || blockToNodeMap.isNew(nodeWithNewStamp)) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   339
                                continue;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   340
                            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   341
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   342
                            Block block = getBlock(nodeWithNewStamp, blockToNodeMap);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   343
                            if (block == null || block.getId() <= mergeBlockDominator.getId()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   344
                                // Node with new stamp in path to the merge block dominator and that
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   345
                                // at the same time was defined at least in the merge block
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   346
                                // dominator (i.e., therefore can be used after the merge.)
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   347
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   348
                                Stamp bestStamp = getBestStamp(nodeWithNewStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   349
                                assert bestStamp != null;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   350
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   351
                                if (currentEndMap != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   352
                                    Stamp otherEndsStamp = currentEndMap.get(nodeWithNewStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   353
                                    if (otherEndsStamp == null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   354
                                        // No stamp registered in one of the previously processed
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   355
                                        // ends => skip.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   356
                                        continue;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   357
                                    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   358
                                    bestStamp = bestStamp.meet(otherEndsStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   359
                                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   360
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   361
                                if (nodeWithNewStamp.stamp(NodeView.DEFAULT).tryImproveWith(bestStamp) == null) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   362
                                    // No point in registering the stamp.
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   363
                                } else {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   364
                                    endMap.put(nodeWithNewStamp, bestStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   365
                                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   366
                            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   367
                        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   368
                        currentBlock = currentBlock.getDominator();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   369
                    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   370
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   371
                    endMaps.put(merge, endMap);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   372
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   373
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   374
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   375
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   376
        private static Block getBlock(ValueNode node, NodeMap<Block> blockToNodeMap) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   377
            if (node instanceof PhiNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   378
                PhiNode phiNode = (PhiNode) node;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   379
                return blockToNodeMap.get(phiNode.merge());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   380
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   381
            return blockToNodeMap.get(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   382
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   383
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   384
        protected void processUnary(UnaryNode node) {
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   385
            ValueNode value = node.getValue();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   386
            Stamp bestStamp = getBestStamp(value);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   387
            Stamp newStamp = node.foldStamp(bestStamp);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   388
            if (!checkReplaceWithConstant(newStamp, node)) {
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   389
                if (!bestStamp.equals(value.stamp(NodeView.DEFAULT))) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   390
                    ValueNode newNode = node.canonical(rawCanonicalizerTool);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   391
                    if (newNode != node) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   392
                        // Canonicalization successfully triggered.
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   393
                        if (newNode != null && !newNode.isAlive()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   394
                            newNode = graph.addWithoutUniqueWithInputs(newNode);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   395
                        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   396
                        node.replaceAndDelete(newNode);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   397
                        GraphUtil.tryKillUnused(value);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   398
                        return;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   399
                    }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   400
                }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   401
                registerNewValueStamp(node, newStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   402
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   403
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   404
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   405
        protected boolean checkReplaceWithConstant(Stamp newStamp, ValueNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   406
            Constant constant = newStamp.asConstant();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   407
            if (constant != null && !(node instanceof ConstantNode)) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   408
                ConstantNode stampConstant = ConstantNode.forConstant(newStamp, constant, metaAccess, graph);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   409
                debug.log("RawConditionElimination: constant stamp replaces %1s with %1s", node, stampConstant);
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   410
                counterConstantReplacements.increment(debug);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   411
                node.replaceAtUsages(InputType.Value, stampConstant);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   412
                GraphUtil.tryKillUnused(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   413
                return true;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   414
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   415
            return false;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   416
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   417
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   418
        protected void processBinary(BinaryNode node) {
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   419
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   420
            ValueNode x = node.getX();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   421
            ValueNode y = node.getY();
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   422
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   423
            Stamp xStamp = getBestStamp(x);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   424
            Stamp yStamp = getBestStamp(y);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   425
            Stamp newStamp = node.foldStamp(xStamp, yStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   426
            if (!checkReplaceWithConstant(newStamp, node)) {
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   427
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   428
                if (!xStamp.equals(x.stamp(NodeView.DEFAULT)) || !yStamp.equals(y.stamp(NodeView.DEFAULT))) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   429
                    // At least one of the inputs has an improved stamp => attempt to canonicalize
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   430
                    // based on that improvement.
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   431
                    ValueNode newNode = node.canonical(rawCanonicalizerTool);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   432
                    if (newNode != node) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   433
                        // Canonicalization successfully triggered.
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   434
                        if (newNode != null && !newNode.isAlive()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   435
                            newNode = graph.addWithoutUniqueWithInputs(newNode);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   436
                        }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   437
                        node.replaceAndDelete(newNode);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   438
                        GraphUtil.tryKillUnused(x);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   439
                        GraphUtil.tryKillUnused(y);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   440
                        return;
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   441
                    }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   442
                }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   443
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   444
                registerNewValueStamp(node, newStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   445
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   446
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   447
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   448
        protected void processIntegerSwitch(IntegerSwitchNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   449
            Stamp bestStamp = getBestStamp(node.value());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   450
            if (node.tryRemoveUnreachableKeys(null, bestStamp)) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   451
                debug.log("\t Canonicalized integer switch %s for value %s and stamp %s", node, node.value(), bestStamp);
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   452
                counterCanonicalizedSwitches.increment(debug);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   453
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   454
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   455
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   456
        protected void processIf(IfNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   457
            TriState result = tryProveCondition(node.condition());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   458
            if (result != TriState.UNKNOWN) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   459
                boolean isTrue = (result == TriState.TRUE);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   460
                AbstractBeginNode survivingSuccessor = node.getSuccessor(isTrue);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   461
                survivingSuccessor.replaceAtUsages(null);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   462
                survivingSuccessor.replaceAtPredecessor(null);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   463
                node.replaceAtPredecessor(survivingSuccessor);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   464
                GraphUtil.killCFG(node);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   465
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   466
                counterIfsKilled.increment(debug);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   467
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   468
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   469
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   470
        protected void processConditional(ConditionalNode node) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   471
            TriState result = tryProveCondition(node.condition());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   472
            if (result != TriState.UNKNOWN) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   473
                boolean isTrue = (result == TriState.TRUE);
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   474
                counterConditionalsKilled.increment(debug);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   475
                node.replaceAndDelete(isTrue ? node.trueValue() : node.falseValue());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   476
            } else {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   477
                Stamp trueStamp = getBestStamp(node.trueValue());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   478
                Stamp falseStamp = getBestStamp(node.falseValue());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   479
                registerNewStamp(node, trueStamp.meet(falseStamp));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   480
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   481
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   482
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   483
        protected TriState tryProveCondition(LogicNode condition) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   484
            Stamp conditionStamp = this.getBestStamp(condition);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   485
            if (conditionStamp == StampFactory.tautology()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   486
                return TriState.TRUE;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   487
            } else if (conditionStamp == StampFactory.contradiction()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   488
                return TriState.FALSE;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   489
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   490
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   491
            if (condition instanceof UnaryOpLogicNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   492
                UnaryOpLogicNode unaryOpLogicNode = (UnaryOpLogicNode) condition;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   493
                return unaryOpLogicNode.tryFold(this.getBestStamp(unaryOpLogicNode.getValue()));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   494
            } else if (condition instanceof BinaryOpLogicNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   495
                BinaryOpLogicNode binaryOpLogicNode = (BinaryOpLogicNode) condition;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   496
                return binaryOpLogicNode.tryFold(this.getBestStamp(binaryOpLogicNode.getX()), this.getBestStamp(binaryOpLogicNode.getY()));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   497
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   498
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   499
            return TriState.UNKNOWN;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   500
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   501
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   502
        protected void processAbstractBegin(AbstractBeginNode beginNode) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   503
            Node predecessor = beginNode.predecessor();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   504
            if (predecessor instanceof IfNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   505
                IfNode ifNode = (IfNode) predecessor;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   506
                boolean negated = (ifNode.falseSuccessor() == beginNode);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   507
                LogicNode condition = ifNode.condition();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   508
                registerNewCondition(condition, negated);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   509
            } else if (predecessor instanceof IntegerSwitchNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   510
                IntegerSwitchNode integerSwitchNode = (IntegerSwitchNode) predecessor;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   511
                registerIntegerSwitch(beginNode, integerSwitchNode);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   512
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   513
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   514
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   515
        private void registerIntegerSwitch(AbstractBeginNode beginNode, IntegerSwitchNode integerSwitchNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   516
            registerNewValueStamp(integerSwitchNode.value(), integerSwitchNode.getValueStampForSuccessor(beginNode));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   517
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   518
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   519
        protected void registerNewCondition(LogicNode condition, boolean negated) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   520
            if (condition instanceof UnaryOpLogicNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   521
                UnaryOpLogicNode unaryLogicNode = (UnaryOpLogicNode) condition;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   522
                ValueNode value = unaryLogicNode.getValue();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   523
                Stamp newStamp = unaryLogicNode.getSucceedingStampForValue(negated);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   524
                registerNewValueStamp(value, newStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   525
            } else if (condition instanceof BinaryOpLogicNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   526
                BinaryOpLogicNode binaryOpLogicNode = (BinaryOpLogicNode) condition;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   527
                ValueNode x = binaryOpLogicNode.getX();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   528
                ValueNode y = binaryOpLogicNode.getY();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   529
                Stamp xStamp = getBestStamp(x);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   530
                Stamp yStamp = getBestStamp(y);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   531
                registerNewValueStamp(x, binaryOpLogicNode.getSucceedingStampForX(negated, xStamp, yStamp));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   532
                registerNewValueStamp(y, binaryOpLogicNode.getSucceedingStampForY(negated, xStamp, yStamp));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   533
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   534
            registerCondition(condition, negated);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   535
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   536
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   537
        protected void registerCondition(LogicNode condition, boolean negated) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   538
            registerNewStamp(condition, negated ? StampFactory.contradiction() : StampFactory.tautology());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   539
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   540
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   541
        protected boolean registerNewValueStamp(ValueNode value, Stamp newStamp) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   542
            if (newStamp != null && !value.isConstant()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   543
                Stamp currentStamp = getBestStamp(value);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   544
                Stamp betterStamp = currentStamp.tryImproveWith(newStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   545
                if (betterStamp != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   546
                    registerNewStamp(value, betterStamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   547
                    return true;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   548
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   549
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   550
            return false;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   551
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   552
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   553
        protected void registerNewStamp(ValueNode value, Stamp newStamp) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   554
            counterStampsRegistered.increment(debug);
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   555
            debug.log("\t Saving stamp for node %s stamp %s", value, newStamp);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   556
            ValueNode originalNode = value;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   557
            stampMap.setAndGrow(originalNode, new StampElement(newStamp, stampMap.getAndGrow(originalNode)));
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   558
            undoOperations.push(originalNode);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   559
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   560
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   561
        protected Stamp getBestStamp(ValueNode value) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   562
            ValueNode originalNode = value;
54084
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   563
            if (!value.isAlive()) {
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   564
                return value.stamp(NodeView.DEFAULT);
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   565
            }
84f10bbf993f 8218074: Update Graal
jwilhelm
parents: 52910
diff changeset
   566
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   567
            StampElement currentStamp = stampMap.getAndGrow(originalNode);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   568
            if (currentStamp == null) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47798
diff changeset
   569
                return value.stamp(NodeView.DEFAULT);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   570
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   571
            return currentStamp.getStamp();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   572
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   573
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   574
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   575
        public Integer enter(Block b) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   576
            int mark = undoOperations.size();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   577
            blockActionStart.put(b, mark);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   578
            for (Node n : schedule.getBlockToNodesMap().get(b)) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   579
                if (n.isAlive()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   580
                    processNode(n);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   581
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   582
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   583
            return mark;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   584
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   585
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   586
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   587
        public void exit(Block b, Integer state) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   588
            int mark = state;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   589
            while (undoOperations.size() > mark) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   590
                Node node = undoOperations.pop();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   591
                if (node.isAlive()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   592
                    stampMap.set(node, stampMap.get(node).getParent());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   593
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   594
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   595
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   596
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   597
    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   598
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   599
    public FixReadsPhase(boolean replaceInputsWithConstants, Phase schedulePhase) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   600
        this.replaceInputsWithConstants = replaceInputsWithConstants;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   601
        this.schedulePhase = schedulePhase;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   602
    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   603
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   604
    @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   605
    protected void run(StructuredGraph graph, LowTierContext context) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   606
        schedulePhase.apply(graph);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   607
        ScheduleResult schedule = graph.getLastSchedule();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   608
        FixReadsClosure fixReadsClosure = new FixReadsClosure();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   609
        for (Block block : schedule.getCFG().getBlocks()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   610
            fixReadsClosure.processNodes(block, schedule);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   611
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   612
        if (GraalOptions.RawConditionalElimination.getValue(graph.getOptions())) {
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   613
            schedule.getCFG().visitDominatorTree(createVisitor(graph, schedule, context), false);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   614
        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   615
        graph.setAfterFixReadPhase(true);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   616
    }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   617
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   618
    public static class RawCEPhase extends BasePhase<LowTierContext> {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   619
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   620
        private final boolean replaceInputsWithConstants;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   621
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   622
        public RawCEPhase(boolean replaceInputsWithConstants) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   623
            this.replaceInputsWithConstants = replaceInputsWithConstants;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   624
        }
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   625
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   626
        @Override
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   627
        protected CharSequence getName() {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   628
            return "RawCEPhase";
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   629
        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   630
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   631
        @Override
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   632
        protected void run(StructuredGraph graph, LowTierContext context) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   633
            if (GraalOptions.RawConditionalElimination.getValue(graph.getOptions())) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   634
                SchedulePhase schedulePhase = new SchedulePhase(SchedulingStrategy.LATEST, true);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   635
                schedulePhase.apply(graph);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   636
                ScheduleResult schedule = graph.getLastSchedule();
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   637
                schedule.getCFG().visitDominatorTree(new RawConditionalEliminationVisitor(graph, schedule, context.getMetaAccess(), replaceInputsWithConstants), false);
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   638
            }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   639
        }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   640
    }
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   641
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents: 54084
diff changeset
   642
    protected ControlFlowGraph.RecursiveVisitor<?> createVisitor(StructuredGraph graph, ScheduleResult schedule, CoreProviders context) {
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   643
        return new RawConditionalEliminationVisitor(graph, schedule, context.getMetaAccess(), replaceInputsWithConstants);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   644
    }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   645
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   646
    protected static final class StampElement {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   647
        private final Stamp stamp;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   648
        private final StampElement parent;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   649
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   650
        public StampElement(Stamp stamp, StampElement parent) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   651
            this.stamp = stamp;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   652
            this.parent = parent;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   653
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   654
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   655
        public StampElement getParent() {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   656
            return parent;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   657
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   658
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   659
        public Stamp getStamp() {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   660
            return stamp;
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   661
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   662
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   663
        @Override
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   664
        public String toString() {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   665
            StringBuilder result = new StringBuilder();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   666
            result.append(stamp);
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   667
            if (this.parent != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   668
                result.append(" (");
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   669
                result.append(this.parent.toString());
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   670
                result.append(")");
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   671
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   672
            return result.toString();
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   673
        }
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   674
    }
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   675
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   676
    public void setReplaceInputsWithConstants(boolean replaceInputsWithConstants) {
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   677
        this.replaceInputsWithConstants = replaceInputsWithConstants;
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46393
diff changeset
   678
    }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents:
diff changeset
   679
}