src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.virtual/src/org/graalvm/compiler/virtual/phases/ea/PEReadEliminationClosure.java
author dlong
Fri, 01 Dec 2017 11:17:45 -0800
changeset 48190 25cfedf27edc
parent 47667 390896759aa2
child 48861 47f19ff9903c
permissions -rw-r--r--
8192814: Update Graal Reviewed-by: kvn
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     1
/*
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     2
 * Copyright (c) 2011, 2016, Oracle and/or its affiliates. All rights reserved.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     4
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     7
 * published by the Free Software Foundation.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     8
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    13
 * accompanied this code).
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    14
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    18
 *
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    21
 * questions.
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    22
 */
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    23
package org.graalvm.compiler.virtual.phases.ea;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    24
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    25
import static org.graalvm.compiler.core.common.GraalOptions.ReadEliminationMaxLoopVisits;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    26
import static org.graalvm.compiler.nodes.NamedLocationIdentity.ARRAY_LENGTH_LOCATION;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    27
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    28
import java.util.EnumMap;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    29
import java.util.Iterator;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    30
import java.util.List;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    31
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    32
import org.graalvm.compiler.core.common.cfg.Loop;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    33
import org.graalvm.compiler.core.common.spi.ConstantFieldProvider;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    34
import org.graalvm.compiler.graph.Node;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    35
import org.graalvm.compiler.nodes.AbstractBeginNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    36
import org.graalvm.compiler.nodes.FieldLocationIdentity;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    37
import org.graalvm.compiler.nodes.FixedNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.FixedWithNextNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.LoopBeginNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    40
import org.graalvm.compiler.nodes.LoopExitNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    41
import org.graalvm.compiler.nodes.NamedLocationIdentity;
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
    42
import org.graalvm.compiler.nodes.NodeView;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    43
import org.graalvm.compiler.nodes.PhiNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    44
import org.graalvm.compiler.nodes.ProxyNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    45
import org.graalvm.compiler.nodes.StructuredGraph.ScheduleResult;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    46
import org.graalvm.compiler.nodes.ValueNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    47
import org.graalvm.compiler.nodes.ValueProxyNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    48
import org.graalvm.compiler.nodes.cfg.Block;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    49
import org.graalvm.compiler.nodes.extended.RawLoadNode;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    50
import org.graalvm.compiler.nodes.extended.RawStoreNode;
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    51
import org.graalvm.compiler.nodes.extended.UnboxNode;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    52
import org.graalvm.compiler.nodes.java.ArrayLengthNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    53
import org.graalvm.compiler.nodes.java.LoadFieldNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    54
import org.graalvm.compiler.nodes.java.LoadIndexedNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    55
import org.graalvm.compiler.nodes.java.StoreFieldNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    56
import org.graalvm.compiler.nodes.java.StoreIndexedNode;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    57
import org.graalvm.compiler.nodes.memory.MemoryCheckpoint;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    58
import org.graalvm.compiler.nodes.spi.LoweringProvider;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    59
import org.graalvm.compiler.nodes.type.StampTool;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    60
import org.graalvm.compiler.nodes.util.GraphUtil;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    61
import org.graalvm.compiler.nodes.virtual.VirtualArrayNode;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    62
import org.graalvm.compiler.options.OptionValues;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    63
import org.graalvm.compiler.virtual.phases.ea.PEReadEliminationBlockState.ReadCacheEntry;
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    64
import org.graalvm.util.EconomicMap;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    65
import org.graalvm.util.EconomicSet;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    66
import org.graalvm.util.Equivalence;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    67
import org.graalvm.util.MapCursor;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    68
import org.graalvm.util.Pair;
46551
d01034a83ab2 8182557: Update Graal
iveresov
parents: 46459
diff changeset
    69
import org.graalvm.word.LocationIdentity;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    70
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    71
import jdk.vm.ci.meta.ConstantReflectionProvider;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    72
import jdk.vm.ci.meta.JavaConstant;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    73
import jdk.vm.ci.meta.JavaKind;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    74
import jdk.vm.ci.meta.MetaAccessProvider;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    75
import jdk.vm.ci.meta.ResolvedJavaType;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    76
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
    77
public final class PEReadEliminationClosure extends PartialEscapeClosure<PEReadEliminationBlockState> {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    78
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    79
    private static final EnumMap<JavaKind, LocationIdentity> UNBOX_LOCATIONS;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    80
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    81
    static {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    82
        UNBOX_LOCATIONS = new EnumMap<>(JavaKind.class);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    83
        for (JavaKind kind : JavaKind.values()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    84
            UNBOX_LOCATIONS.put(kind, NamedLocationIdentity.immutable("PEA unbox " + kind.getJavaName()));
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    85
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    86
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    87
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    88
    public PEReadEliminationClosure(ScheduleResult schedule, MetaAccessProvider metaAccess, ConstantReflectionProvider constantReflection, ConstantFieldProvider constantFieldProvider,
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    89
                    LoweringProvider loweringProvider) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    90
        super(schedule, metaAccess, constantReflection, constantFieldProvider, loweringProvider);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    91
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    92
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    93
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    94
    protected PEReadEliminationBlockState getInitialState() {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
    95
        return new PEReadEliminationBlockState(tool.getOptions(), tool.getDebug());
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    96
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    97
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    98
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
    99
    protected boolean processNode(Node node, PEReadEliminationBlockState state, GraphEffectList effects, FixedWithNextNode lastFixedNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   100
        if (super.processNode(node, state, effects, lastFixedNode)) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   101
            return true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   102
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   103
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   104
        if (node instanceof LoadFieldNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   105
            return processLoadField((LoadFieldNode) node, state, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   106
        } else if (node instanceof StoreFieldNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   107
            return processStoreField((StoreFieldNode) node, state, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   108
        } else if (node instanceof LoadIndexedNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   109
            return processLoadIndexed((LoadIndexedNode) node, state, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   110
        } else if (node instanceof StoreIndexedNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   111
            return processStoreIndexed((StoreIndexedNode) node, state, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   112
        } else if (node instanceof ArrayLengthNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   113
            return processArrayLength((ArrayLengthNode) node, state, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   114
        } else if (node instanceof UnboxNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   115
            return processUnbox((UnboxNode) node, state, effects);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   116
        } else if (node instanceof RawLoadNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   117
            return processUnsafeLoad((RawLoadNode) node, state, effects);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   118
        } else if (node instanceof RawStoreNode) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   119
            return processUnsafeStore((RawStoreNode) node, state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   120
        } else if (node instanceof MemoryCheckpoint.Single) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   121
            COUNTER_MEMORYCHECKPOINT.increment(node.getDebug());
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   122
            LocationIdentity identity = ((MemoryCheckpoint.Single) node).getLocationIdentity();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   123
            processIdentity(state, identity);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   124
        } else if (node instanceof MemoryCheckpoint.Multi) {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   125
            COUNTER_MEMORYCHECKPOINT.increment(node.getDebug());
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   126
            for (LocationIdentity identity : ((MemoryCheckpoint.Multi) node).getLocationIdentities()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   127
                processIdentity(state, identity);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   128
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   129
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   130
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   131
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   132
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   133
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   134
    private boolean processStore(FixedNode store, ValueNode object, LocationIdentity identity, int index, JavaKind accessKind, boolean overflowAccess, ValueNode value,
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   135
                    PEReadEliminationBlockState state, GraphEffectList effects) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   136
        ValueNode unproxiedObject = GraphUtil.unproxify(object);
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   137
        ValueNode cachedValue = state.getReadCache(object, identity, index, accessKind, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   138
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   139
        ValueNode finalValue = getScalarAlias(value);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   140
        boolean result = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   141
        if (GraphUtil.unproxify(finalValue) == GraphUtil.unproxify(cachedValue)) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   142
            effects.deleteNode(store);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   143
            result = true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   144
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   145
        state.killReadCache(identity, index);
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   146
        state.addReadCache(unproxiedObject, identity, index, accessKind, overflowAccess, finalValue, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   147
        return result;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   148
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   149
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   150
    private boolean processLoad(FixedNode load, ValueNode object, LocationIdentity identity, int index, JavaKind kind, PEReadEliminationBlockState state, GraphEffectList effects) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   151
        ValueNode unproxiedObject = GraphUtil.unproxify(object);
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   152
        ValueNode cachedValue = state.getReadCache(unproxiedObject, identity, index, kind, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   153
        if (cachedValue != null) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   154
            // perform the read elimination
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   155
            effects.replaceAtUsages(load, cachedValue, load);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   156
            addScalarAlias(load, cachedValue);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   157
            return true;
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   158
        } else {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   159
            state.addReadCache(unproxiedObject, identity, index, kind, false, load, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   160
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   161
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   162
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   163
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   164
    private static boolean isOverflowAccess(JavaKind accessKind, JavaKind declaredKind) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   165
        if (accessKind == declaredKind) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   166
            return false;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   167
        }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   168
        if (accessKind == JavaKind.Object) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   169
            switch (declaredKind) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   170
                case Object:
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   171
                case Double:
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   172
                case Long:
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   173
                    return false;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   174
                default:
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   175
                    return true;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   176
            }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   177
        }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   178
        assert accessKind.isPrimitive() : "Illegal access kind";
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   179
        return declaredKind.isPrimitive() ? accessKind.getBitCount() > declaredKind.getBitCount() : true;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   180
    }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   181
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   182
    private boolean processUnsafeLoad(RawLoadNode load, PEReadEliminationBlockState state, GraphEffectList effects) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   183
        if (load.offset().isConstant()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   184
            ResolvedJavaType type = StampTool.typeOrNull(load.object());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   185
            if (type != null && type.isArray()) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   186
                JavaKind accessKind = load.accessKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   187
                JavaKind componentKind = type.getComponentType().getJavaKind();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   188
                long offset = load.offset().asJavaConstant().asLong();
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   189
                int index = VirtualArrayNode.entryIndexForOffset(offset, accessKind, type.getComponentType(), Integer.MAX_VALUE);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   190
                ValueNode object = GraphUtil.unproxify(load.object());
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   191
                LocationIdentity location = NamedLocationIdentity.getArrayLocation(componentKind);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   192
                ValueNode cachedValue = state.getReadCache(object, location, index, accessKind, this);
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   193
                assert cachedValue == null || load.stamp(NodeView.DEFAULT).isCompatible(cachedValue.stamp(NodeView.DEFAULT)) : "The RawLoadNode's stamp is not compatible with the cached value.";
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   194
                if (cachedValue != null) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   195
                    effects.replaceAtUsages(load, cachedValue, load);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   196
                    addScalarAlias(load, cachedValue);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   197
                    return true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   198
                } else {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   199
                    state.addReadCache(object, location, index, accessKind, isOverflowAccess(accessKind, componentKind), load, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   200
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   201
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   202
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   203
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   204
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   205
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   206
    private boolean processUnsafeStore(RawStoreNode store, PEReadEliminationBlockState state, GraphEffectList effects) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   207
        ResolvedJavaType type = StampTool.typeOrNull(store.object());
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   208
        if (type != null && type.isArray()) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   209
            JavaKind accessKind = store.accessKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   210
            JavaKind componentKind = type.getComponentType().getJavaKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   211
            LocationIdentity location = NamedLocationIdentity.getArrayLocation(componentKind);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   212
            if (store.offset().isConstant()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   213
                long offset = store.offset().asJavaConstant().asLong();
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   214
                boolean overflowAccess = isOverflowAccess(accessKind, componentKind);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   215
                int index = overflowAccess ? -1 : VirtualArrayNode.entryIndexForOffset(offset, accessKind, type.getComponentType(), Integer.MAX_VALUE);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   216
                return processStore(store, store.object(), location, index, accessKind, overflowAccess, store.value(), state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   217
            } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   218
                processIdentity(state, location);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   219
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   220
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   221
            state.killReadCache();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   222
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   223
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   224
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   225
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   226
    private boolean processArrayLength(ArrayLengthNode length, PEReadEliminationBlockState state, GraphEffectList effects) {
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   227
        return processLoad(length, length.array(), ARRAY_LENGTH_LOCATION, -1, JavaKind.Int, state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   228
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   229
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   230
    private boolean processStoreField(StoreFieldNode store, PEReadEliminationBlockState state, GraphEffectList effects) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   231
        if (store.isVolatile()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   232
            state.killReadCache();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   233
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   234
        }
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   235
        JavaKind kind = store.field().getJavaKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   236
        return processStore(store, store.object(), new FieldLocationIdentity(store.field()), -1, kind, false, store.value(), state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   237
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   238
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   239
    private boolean processLoadField(LoadFieldNode load, PEReadEliminationBlockState state, GraphEffectList effects) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   240
        if (load.isVolatile()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   241
            state.killReadCache();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   242
            return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   243
        }
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   244
        return processLoad(load, load.object(), new FieldLocationIdentity(load.field()), -1, load.field().getJavaKind(), state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   245
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   246
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   247
    private static JavaKind getElementKindFromStamp(ValueNode array) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   248
        ResolvedJavaType type = StampTool.typeOrNull(array);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   249
        if (type != null && type.isArray()) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   250
            return type.getComponentType().getJavaKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   251
        } else {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   252
            // It is likely an OSRLocal without valid stamp
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   253
            return JavaKind.Illegal;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   254
        }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   255
    }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   256
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   257
    private boolean processStoreIndexed(StoreIndexedNode store, PEReadEliminationBlockState state, GraphEffectList effects) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   258
        int index = store.index().isConstant() ? ((JavaConstant) store.index().asConstant()).asInt() : -1;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   259
        // BASTORE (with elementKind being Byte) can be used to store values in boolean arrays.
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   260
        JavaKind elementKind = store.elementKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   261
        if (elementKind == JavaKind.Byte) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   262
            elementKind = getElementKindFromStamp(store.array());
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   263
            if (elementKind == JavaKind.Illegal) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   264
                // Could not determine the actual access kind from stamp. Hence kill both.
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   265
                state.killReadCache(NamedLocationIdentity.getArrayLocation(JavaKind.Boolean), index);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   266
                state.killReadCache(NamedLocationIdentity.getArrayLocation(JavaKind.Byte), index);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   267
                return false;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   268
            }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   269
        }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   270
        LocationIdentity arrayLocation = NamedLocationIdentity.getArrayLocation(elementKind);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   271
        if (index != -1) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   272
            return processStore(store, store.array(), arrayLocation, index, elementKind, false, store.value(), state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   273
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   274
            state.killReadCache(arrayLocation, -1);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   275
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   276
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   277
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   278
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   279
    private boolean processLoadIndexed(LoadIndexedNode load, PEReadEliminationBlockState state, GraphEffectList effects) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   280
        if (load.index().isConstant()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   281
            int index = ((JavaConstant) load.index().asConstant()).asInt();
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   282
            // BALOAD (with elementKind being Byte) can be used to retrieve values from boolean
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   283
            // arrays.
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   284
            JavaKind elementKind = load.elementKind();
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   285
            if (elementKind == JavaKind.Byte) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   286
                elementKind = getElementKindFromStamp(load.array());
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   287
                if (elementKind == JavaKind.Illegal) {
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   288
                    return false;
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   289
                }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   290
            }
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   291
            LocationIdentity arrayLocation = NamedLocationIdentity.getArrayLocation(elementKind);
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   292
            return processLoad(load, load.array(), arrayLocation, index, elementKind, state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   293
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   294
        return false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   295
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   296
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   297
    private boolean processUnbox(UnboxNode unbox, PEReadEliminationBlockState state, GraphEffectList effects) {
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   298
        return processLoad(unbox, unbox.getValue(), UNBOX_LOCATIONS.get(unbox.getBoxingKind()), -1, unbox.getBoxingKind(), state, effects);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   299
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   300
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   301
    private static void processIdentity(PEReadEliminationBlockState state, LocationIdentity identity) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   302
        if (identity.isAny()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   303
            state.killReadCache();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   304
        } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   305
            state.killReadCache(identity, -1);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   306
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   307
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   308
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   309
    @SuppressWarnings("unchecked")
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   310
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   311
    protected void processInitialLoopState(Loop<Block> loop, PEReadEliminationBlockState initialState) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   312
        super.processInitialLoopState(loop, initialState);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   313
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   314
        if (!initialState.getReadCache().isEmpty()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   315
            EconomicMap<ValueNode, Pair<ValueNode, Object>> firstValueSet = null;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   316
            for (PhiNode phi : ((LoopBeginNode) loop.getHeader().getBeginNode()).phis()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   317
                ValueNode firstValue = phi.valueAt(0);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   318
                if (firstValue != null && phi.getStackKind().isObject()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   319
                    ValueNode unproxified = GraphUtil.unproxify(firstValue);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   320
                    if (firstValueSet == null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   321
                        firstValueSet = EconomicMap.create(Equivalence.IDENTITY_WITH_SYSTEM_HASHCODE);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   322
                    }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   323
                    Pair<ValueNode, Object> pair = Pair.create(unproxified, firstValueSet.get(unproxified));
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   324
                    firstValueSet.put(unproxified, pair);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   325
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   326
            }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   327
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   328
            if (firstValueSet != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   329
                ReadCacheEntry[] entries = new ReadCacheEntry[initialState.getReadCache().size()];
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   330
                int z = 0;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   331
                for (ReadCacheEntry entry : initialState.getReadCache().getKeys()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   332
                    entries[z++] = entry;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   333
                }
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   334
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   335
                for (ReadCacheEntry entry : entries) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   336
                    ValueNode object = entry.object;
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   337
                    if (object != null) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   338
                        Pair<ValueNode, Object> pair = firstValueSet.get(object);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   339
                        while (pair != null) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   340
                            initialState.addReadCache(pair.getLeft(), entry.identity, entry.index, entry.kind, entry.overflowAccess, initialState.getReadCache().get(entry), this);
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   341
                            pair = (Pair<ValueNode, Object>) pair.getRight();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   342
                        }
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   343
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   344
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   345
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   346
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   347
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   348
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   349
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   350
    protected void processLoopExit(LoopExitNode exitNode, PEReadEliminationBlockState initialState, PEReadEliminationBlockState exitState, GraphEffectList effects) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   351
        super.processLoopExit(exitNode, initialState, exitState, effects);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   352
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   353
        if (exitNode.graph().hasValueProxies()) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   354
            MapCursor<ReadCacheEntry, ValueNode> entry = exitState.getReadCache().getEntries();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   355
            while (entry.advance()) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   356
                if (initialState.getReadCache().get(entry.getKey()) != entry.getValue()) {
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   357
                    ValueNode value = exitState.getReadCache(entry.getKey().object, entry.getKey().identity, entry.getKey().index, entry.getKey().kind, this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   358
                    assert value != null : "Got null from read cache, entry's value:" + entry.getValue();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   359
                    if (!(value instanceof ProxyNode) || ((ProxyNode) value).proxyPoint() != exitNode) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   360
                        ProxyNode proxy = new ValueProxyNode(value, exitNode);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   361
                        effects.addFloatingNode(proxy, "readCacheProxy");
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   362
                        exitState.getReadCache().put(entry.getKey(), proxy);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   363
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   364
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   365
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   366
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   367
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   368
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   369
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   370
    protected PEReadEliminationBlockState cloneState(PEReadEliminationBlockState other) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   371
        return new PEReadEliminationBlockState(other);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   372
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   373
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   374
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   375
    protected MergeProcessor createMergeProcessor(Block merge) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   376
        return new ReadEliminationMergeProcessor(merge);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   377
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   378
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   379
    private class ReadEliminationMergeProcessor extends MergeProcessor {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   380
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   381
        ReadEliminationMergeProcessor(Block mergeBlock) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   382
            super(mergeBlock);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   383
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   384
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   385
        @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   386
        protected void merge(List<PEReadEliminationBlockState> states) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   387
            super.merge(states);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   388
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   389
            mergeReadCache(states);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   390
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   391
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   392
        private void mergeReadCache(List<PEReadEliminationBlockState> states) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   393
            MapCursor<ReadCacheEntry, ValueNode> cursor = states.get(0).readCache.getEntries();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   394
            while (cursor.advance()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   395
                ReadCacheEntry key = cursor.getKey();
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   396
                ValueNode value = cursor.getValue();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   397
                boolean phi = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   398
                for (int i = 1; i < states.size(); i++) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   399
                    ValueNode otherValue = states.get(i).readCache.get(key);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   400
                    // e.g. unsafe loads / stores with different access kinds have different stamps
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   401
                    // although location, object and offset are the same, in this case we cannot
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   402
                    // create a phi nor can we set a common value
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   403
                    if (otherValue == null || !value.stamp(NodeView.DEFAULT).isCompatible(otherValue.stamp(NodeView.DEFAULT))) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   404
                        value = null;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   405
                        phi = false;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   406
                        break;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   407
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   408
                    if (!phi && otherValue != value) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   409
                        phi = true;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   410
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   411
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   412
                if (phi) {
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   413
                    PhiNode phiNode = getPhi(key, value.stamp(NodeView.DEFAULT).unrestricted());
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   414
                    mergeEffects.addFloatingNode(phiNode, "mergeReadCache");
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   415
                    for (int i = 0; i < states.size(); i++) {
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   416
                        ValueNode v = states.get(i).getReadCache(key.object, key.identity, key.index, key.kind, PEReadEliminationClosure.this);
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   417
                        assert phiNode.stamp(NodeView.DEFAULT).isCompatible(v.stamp(NodeView.DEFAULT)) : "Cannot create read elimination phi for inputs with incompatible stamps.";
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   418
                        setPhiInput(phiNode, i, v);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   419
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   420
                    newState.readCache.put(key, phiNode);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   421
                } else if (value != null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   422
                    newState.readCache.put(key, value);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   423
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   424
            }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   425
            /*
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   426
             * For object phis, see if there are known reads on all predecessors, for which we could
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   427
             * create new phis.
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   428
             */
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   429
            for (PhiNode phi : getPhis()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   430
                if (phi.getStackKind() == JavaKind.Object) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   431
                    for (ReadCacheEntry entry : states.get(0).readCache.getKeys()) {
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   432
                        if (entry.object == getPhiValueAt(phi, 0)) {
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   433
                            mergeReadCachePhi(phi, entry.identity, entry.index, entry.kind, entry.overflowAccess, states);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   434
                        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   435
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   436
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   437
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   438
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   439
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   440
        private void mergeReadCachePhi(PhiNode phi, LocationIdentity identity, int index, JavaKind kind, boolean overflowAccess, List<PEReadEliminationBlockState> states) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   441
            ValueNode[] values = new ValueNode[states.size()];
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   442
            values[0] = states.get(0).getReadCache(getPhiValueAt(phi, 0), identity, index, kind, PEReadEliminationClosure.this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   443
            if (values[0] != null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   444
                for (int i = 1; i < states.size(); i++) {
46762
f7defa99f173 8185829: Update Graal
dlong
parents: 46640
diff changeset
   445
                    ValueNode value = states.get(i).getReadCache(getPhiValueAt(phi, i), identity, index, kind, PEReadEliminationClosure.this);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   446
                    // e.g. unsafe loads / stores with same identity and different access kinds see
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   447
                    // mergeReadCache(states)
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   448
                    if (value == null || !values[i - 1].stamp(NodeView.DEFAULT).isCompatible(value.stamp(NodeView.DEFAULT))) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   449
                        return;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   450
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   451
                    values[i] = value;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   452
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   453
48190
25cfedf27edc 8192814: Update Graal
dlong
parents: 47667
diff changeset
   454
                PhiNode phiNode = getPhi(new ReadCacheEntry(identity, phi, index, kind, overflowAccess), values[0].stamp(NodeView.DEFAULT).unrestricted());
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   455
                mergeEffects.addFloatingNode(phiNode, "mergeReadCachePhi");
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   456
                for (int i = 0; i < values.length; i++) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   457
                    setPhiInput(phiNode, i, values[i]);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   458
                }
47667
390896759aa2 8187438: Update Graal
dlong
parents: 47216
diff changeset
   459
                newState.readCache.put(new ReadCacheEntry(identity, phi, index, kind, overflowAccess), phiNode);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   460
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   461
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   462
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   463
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   464
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   465
    protected void processKilledLoopLocations(Loop<Block> loop, PEReadEliminationBlockState initialState, PEReadEliminationBlockState mergedStates) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   466
        assert initialState != null;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   467
        assert mergedStates != null;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   468
        if (initialState.readCache.size() > 0) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   469
            LoopKillCache loopKilledLocations = loopLocationKillCache.get(loop);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   470
            // we have fully processed this loop the first time, remember to cache it the next time
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   471
            // it is visited
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   472
            if (loopKilledLocations == null) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   473
                loopKilledLocations = new LoopKillCache(1/* 1.visit */);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   474
                loopLocationKillCache.put(loop, loopKilledLocations);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   475
            } else {
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   476
                AbstractBeginNode beginNode = loop.getHeader().getBeginNode();
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   477
                OptionValues options = beginNode.getOptions();
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   478
                if (loopKilledLocations.visits() > ReadEliminationMaxLoopVisits.getValue(options)) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   479
                    // we have processed the loop too many times, kill all locations so the inner
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   480
                    // loop will never be processed more than once again on visit
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   481
                    loopKilledLocations.setKillsAll();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   482
                } else {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   483
                    // we have fully processed this loop >1 times, update the killed locations
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   484
                    EconomicSet<LocationIdentity> forwardEndLiveLocations = EconomicSet.create(Equivalence.DEFAULT);
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   485
                    for (ReadCacheEntry entry : initialState.readCache.getKeys()) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   486
                        forwardEndLiveLocations.add(entry.identity);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   487
                    }
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   488
                    for (ReadCacheEntry entry : mergedStates.readCache.getKeys()) {
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   489
                        forwardEndLiveLocations.remove(entry.identity);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   490
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   491
                    // every location that is alive before the loop but not after is killed by the
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   492
                    // loop
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   493
                    for (LocationIdentity location : forwardEndLiveLocations) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   494
                        loopKilledLocations.rememberLoopKilledLocation(location);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   495
                    }
46640
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   496
                    if (debug.isLogEnabled() && loopKilledLocations != null) {
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   497
                        debug.log("[Early Read Elimination] Setting loop killed locations of loop at node %s with %s",
70bdce04c59b 8183991: Update Graal
iveresov
parents: 46551
diff changeset
   498
                                        beginNode, forwardEndLiveLocations);
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   499
                    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   500
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   501
                // remember the loop visit
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   502
                loopKilledLocations.visited();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   503
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   504
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   505
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   506
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   507
    @Override
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   508
    protected PEReadEliminationBlockState stripKilledLoopLocations(Loop<Block> loop, PEReadEliminationBlockState originalInitialState) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   509
        PEReadEliminationBlockState initialState = super.stripKilledLoopLocations(loop, originalInitialState);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   510
        LoopKillCache loopKilledLocations = loopLocationKillCache.get(loop);
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   511
        if (loopKilledLocations != null && loopKilledLocations.loopKillsLocations()) {
46344
694c102fd8ed 8177046: Update Graal
iveresov
parents: 43972
diff changeset
   512
            Iterator<ReadCacheEntry> it = initialState.readCache.getKeys().iterator();
43972
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   513
            while (it.hasNext()) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   514
                ReadCacheEntry entry = it.next();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   515
                if (loopKilledLocations.containsLocation(entry.identity)) {
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   516
                    it.remove();
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   517
                }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   518
            }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   519
        }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   520
        return initialState;
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   521
    }
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   522
1ade39b8381b 8174879: Rename jdk.vm.ci to jdk.internal.vm.ci
kvn
parents:
diff changeset
   523
}