Mercurial > hg > truffle

annotate graal/com.oracle.graal.phases/src/com/oracle/graal/phases/verify/VerifyUsageWithEquals.java @ 11959:23ccaa863eda

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
author Doug Simon <doug.simon@oracle.com>
date Thu, 10 Oct 2013 16:14:55 +0200
parents eb2def6529bc
children 2c4aa758ee18
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
1 /*
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
2 * Copyright (c) 2013, Oracle and/or its affiliates. All rights reserved.
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
4 *
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
7 * published by the Free Software Foundation.
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
8 *
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
13 * accompanied this code).
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
14 *
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
18 *
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
21 * questions.
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
22 */
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
23 package com.oracle.graal.phases.verify;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
24
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
25 import com.oracle.graal.api.meta.*;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
26 import com.oracle.graal.nodes.*;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
27 import com.oracle.graal.nodes.calc.*;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
28 import com.oracle.graal.nodes.type.*;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
29 import com.oracle.graal.phases.*;
10937
87d9b5518065 Move VerifyUsageWithEquals phases into HighTier.
Roland Schatz <roland.schatz@oracle.com>
parents: 9868
diff changeset
30 import com.oracle.graal.phases.tiers.*;
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
31
9800
2eaac2c3670b VerifyUsageWithEquals/VerifyPhase: add documentation
Bernhard Urban <bernhard.urban@jku.at>
parents: 9799
diff changeset
32 /**
2eaac2c3670b VerifyUsageWithEquals/VerifyPhase: add documentation
Bernhard Urban <bernhard.urban@jku.at>
parents: 9799
diff changeset
33 * For certain types object identity should not be used for object equality check. This phase checks
2eaac2c3670b VerifyUsageWithEquals/VerifyPhase: add documentation
Bernhard Urban <bernhard.urban@jku.at>
parents: 9799
diff changeset
34 * the correct usage of the given type. Equality checks with == or != (except null checks) results
2eaac2c3670b VerifyUsageWithEquals/VerifyPhase: add documentation
Bernhard Urban <bernhard.urban@jku.at>
parents: 9799
diff changeset
35 * in an {@link AssertionError}.
2eaac2c3670b VerifyUsageWithEquals/VerifyPhase: add documentation
Bernhard Urban <bernhard.urban@jku.at>
parents: 9799
diff changeset
36 */
10937
87d9b5518065 Move VerifyUsageWithEquals phases into HighTier.
Roland Schatz <roland.schatz@oracle.com>
parents: 9868
diff changeset
37 public class VerifyUsageWithEquals extends VerifyPhase<PhaseContext> {
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
38
10937
87d9b5518065 Move VerifyUsageWithEquals phases into HighTier.
Roland Schatz <roland.schatz@oracle.com>
parents: 9868
diff changeset
39 private final Class<?> klass;
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
40
10937
87d9b5518065 Move VerifyUsageWithEquals phases into HighTier.
Roland Schatz <roland.schatz@oracle.com>
parents: 9868
diff changeset
41 public VerifyUsageWithEquals(Class<?> klass) {
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
42 this.klass = klass;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
43 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
44
11959
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
45 private boolean isAssignableType(ValueNode node, MetaAccessProvider metaAccess) {
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
46 if (node.stamp() instanceof ObjectStamp) {
11959
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
47 ResolvedJavaType valueType = metaAccess.lookupJavaType(klass);
11265
ef6915cf1e59 Add illegal stamp
Gilles Duboscq <duboscq@ssw.jku.at>
parents: 10937
diff changeset
48 ResolvedJavaType nodeType = ObjectStamp.typeOrNull(node);
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
49
9799
41de6947dbc9 VerifyUsageWithEquals: check for null before passing it to isAssignableForm()
Bernhard Urban <bernhard.urban@jku.at>
parents: 9789
diff changeset
50 if (nodeType != null && valueType.isAssignableFrom(nodeType)) {
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
51 return true;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
52 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
53 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
54 return false;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
55 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
56
9789
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
57 private static boolean isNullConstant(ValueNode node) {
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
58 return node.isConstant() && node.asConstant().isNull();
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
59 }
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
60
11959
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
61 private boolean checkUsage(ValueNode x, ValueNode y, MetaAccessProvider metaAccess) {
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
62 return isAssignableType(x, metaAccess) && !isNullConstant(y);
9789
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
63 }
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
64
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
65 private static boolean isEqualsMethod(StructuredGraph graph) {
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
66 Signature signature = graph.method().getSignature();
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
67 return graph.method().getName().equals("equals") && signature.getParameterCount(false) == 1 && signature.getParameterKind(0).equals(Kind.Object);
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
68 }
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
69
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
70 @Override
10937
87d9b5518065 Move VerifyUsageWithEquals phases into HighTier.
Roland Schatz <roland.schatz@oracle.com>
parents: 9868
diff changeset
71 protected boolean verify(StructuredGraph graph, PhaseContext context) {
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
72 for (ObjectEqualsNode cn : graph.getNodes().filter(ObjectEqualsNode.class)) {
9789
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
73 if (!isEqualsMethod(graph)) {
ad50389f1da4 VerifyUsageWithEquals: refactoring
Bernhard Urban <bernhard.urban@jku.at>
parents: 9788
diff changeset
74 // bail out if we compare an object of type klass with == or != (except null checks)
11959
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
75 assert !(checkUsage(cn.x(), cn.y(), context.getMetaAccess()) && checkUsage(cn.y(), cn.x(), context.getMetaAccess())) : "Verifcation of " + klass.getName() +
23ccaa863eda made CodeCacheProvider independent of MetaAccessProvider (GRAAL-511)
Doug Simon <doug.simon@oracle.com>
parents: 11803
diff changeset
76 " usage failed: Comparing " + cn.x() + " and " + cn.y() + " in " + graph.method() + " must use .equals() for object equality, not '==' or '!='";
9788
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
77 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
78 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
79 return true;
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
80 }
1bd67c5bdab8 VerifyUsageWithEquals: allow to pass a class to the phase
Bernhard Urban <bernhard.urban@jku.at>
parents:
diff changeset
81 }