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