src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.core.test/src/org/graalvm/compiler/core/test/VerifyVirtualizableUsage.java
author chegar
Thu, 17 Oct 2019 20:54:25 +0100
branchdatagramsocketimpl-branch
changeset 58679 9c3209ff7550
parent 58678 9cf78a70fa4f
parent 58299 6df94ce3ab2f
permissions -rw-r--r--
datagramsocketimpl-branch: merge with default
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
52910
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     1
/*
58299
6df94ce3ab2f 8229201: Update Graal
dlong
parents: 55509
diff changeset
     2
 * Copyright (c) 2015, 2019, Oracle and/or its affiliates. All rights reserved.
52910
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     4
 *
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     7
 * published by the Free Software Foundation.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     8
 *
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    13
 * accompanied this code).
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    14
 *
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    18
 *
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    21
 * questions.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    22
 */
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    23
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    24
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    25
package org.graalvm.compiler.core.test;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    26
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    27
import org.graalvm.compiler.core.common.type.ObjectStamp;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    28
import org.graalvm.compiler.core.common.type.Stamp;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    29
import org.graalvm.compiler.graph.Graph;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    30
import org.graalvm.compiler.graph.Node;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    31
import org.graalvm.compiler.graph.NodeInputList;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    32
import org.graalvm.compiler.nodes.ConstantNode;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    33
import org.graalvm.compiler.nodes.NodeView;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    34
import org.graalvm.compiler.nodes.StructuredGraph;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    35
import org.graalvm.compiler.nodes.ValueNode;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    36
import org.graalvm.compiler.nodes.java.MethodCallTargetNode;
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents: 54328
diff changeset
    37
import org.graalvm.compiler.nodes.spi.CoreProviders;
52910
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.spi.Virtualizable;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    39
import org.graalvm.compiler.phases.VerifyPhase;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    40
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    41
import jdk.vm.ci.meta.ResolvedJavaMethod;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    42
import jdk.vm.ci.meta.ResolvedJavaType;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    43
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    44
/**
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    45
 * Verifies that node types implementing the {@link Virtualizable} interface use it correctly.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    46
 * Implementors of {@link Virtualizable#virtualize(org.graalvm.compiler.nodes.spi.VirtualizerTool)}
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    47
 * must not apply effects on their {@link Graph graph} that cannot be easily undone.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    48
 */
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents: 54328
diff changeset
    49
public class VerifyVirtualizableUsage extends VerifyPhase<CoreProviders> {
52910
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    50
    @Override
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    51
    public boolean checkContract() {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    52
        return false;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    53
    }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    54
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    55
    @Override
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents: 54328
diff changeset
    56
    protected void verify(StructuredGraph graph, CoreProviders context) {
52910
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    57
        final ResolvedJavaType graphType = context.getMetaAccess().lookupJavaType(Graph.class);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    58
        final ResolvedJavaType virtualizableType = context.getMetaAccess().lookupJavaType(Virtualizable.class);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    59
        final ResolvedJavaType constantNodeType = context.getMetaAccess().lookupJavaType(ConstantNode.class);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    60
        if (virtualizableType.isAssignableFrom(graph.method().getDeclaringClass()) && graph.method().getName().equals("virtualize")) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    61
            for (MethodCallTargetNode t : graph.getNodes(MethodCallTargetNode.TYPE)) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    62
                int bci = t.invoke().bci();
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    63
                ResolvedJavaMethod callee = t.targetMethod();
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    64
                String calleeName = callee.getName();
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    65
                if (callee.getDeclaringClass().equals(graphType)) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    66
                    if (calleeName.equals("add") || calleeName.equals("addWithoutUnique") || calleeName.equals("addOrUnique") || calleeName.equals("addWithoutUniqueWithInputs") ||
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    67
                                    calleeName.equals("addOrUniqueWithInputs")) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    68
                        verifyVirtualizableEffectArguments(constantNodeType, graph.method(), callee, bci, t.arguments(), 1);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    69
                    }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    70
                }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    71
            }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    72
        }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    73
    }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    74
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    75
    private static void verifyVirtualizableEffectArguments(ResolvedJavaType constantNodeType, ResolvedJavaMethod caller, ResolvedJavaMethod callee, int bciCaller,
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    76
                    NodeInputList<? extends Node> arguments, int startIdx) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    77
        /*
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    78
         * Virtualizable.virtualize should never apply effects on the graph during the execution of
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    79
         * the call as the handling of loops during pea might be speculative and does not hold. We
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    80
         * should only allow nodes changing the graph that do no harm like constants.
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    81
         */
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    82
        int i = 0;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    83
        for (Node arg : arguments) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    84
            if (i >= startIdx) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    85
                Stamp argStamp = ((ValueNode) arg).stamp(NodeView.DEFAULT);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    86
                if (argStamp instanceof ObjectStamp) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    87
                    ObjectStamp objectStamp = (ObjectStamp) argStamp;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    88
                    ResolvedJavaType argStampType = objectStamp.type();
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    89
                    if (!(argStampType.equals(constantNodeType))) {
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    90
                        StackTraceElement e = caller.asStackTraceElement(bciCaller);
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    91
                        throw new VerificationError("%s:Parameter %d in call to %s (which has effects on the graph) is not a " +
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    92
                                        "constant and thus not safe to apply during speculative virtualization.", e, i, callee.format("%H.%n(%p)"));
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    93
                    }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    94
                }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    95
            }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    96
            i++;
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    97
        }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    98
    }
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
    99
583fd71c47d6 8214023: Update Graal
dlong
parents:
diff changeset
   100
}