langtools/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/TransTypes.java
author mcimadamore
Fri, 09 Jan 2015 15:51:20 +0000
changeset 28339 fa0685d84b74
parent 28337 0b46fb7be75a
child 28800 35f9042d53b3
permissions -rw-r--r--
8067914: Redundant type cast nodes in AST (follow up from JDK-8043741) Summary: Workaround for redundant (but harmless) synthetic cast generated in TransTypes Reviewed-by: mcimadamore Contributed-by: srikanth.adayapalam@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     1
/*
23127
5aba53c2b9b2 8034048: javac crash with method references plus lambda plus var args
jjg
parents: 22168
diff changeset
     2
 * Copyright (c) 1999, 2014, Oracle and/or its affiliates. All rights reserved.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
06bc494ca11e Initial load
duke
parents:
diff changeset
     4
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
06bc494ca11e Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 5008
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
10
06bc494ca11e Initial load
duke
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 5008
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    10
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
06bc494ca11e Initial load
duke
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
06bc494ca11e Initial load
duke
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
06bc494ca11e Initial load
duke
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
06bc494ca11e Initial load
duke
parents:
diff changeset
    15
 * accompanied this code).
06bc494ca11e Initial load
duke
parents:
diff changeset
    16
 *
06bc494ca11e Initial load
duke
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
06bc494ca11e Initial load
duke
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
06bc494ca11e Initial load
duke
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
06bc494ca11e Initial load
duke
parents:
diff changeset
    20
 *
5520
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 5008
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 5008
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
86e4b9a9da40 6943119: Rebrand source copyright notices
ohair
parents: 5008
diff changeset
    23
 * questions.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    24
 */
06bc494ca11e Initial load
duke
parents:
diff changeset
    25
06bc494ca11e Initial load
duke
parents:
diff changeset
    26
package com.sun.tools.javac.comp;
06bc494ca11e Initial load
duke
parents:
diff changeset
    27
06bc494ca11e Initial load
duke
parents:
diff changeset
    28
import java.util.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    29
06bc494ca11e Initial load
duke
parents:
diff changeset
    30
import com.sun.tools.javac.code.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    31
import com.sun.tools.javac.code.Symbol.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    32
import com.sun.tools.javac.tree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    33
import com.sun.tools.javac.tree.JCTree.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    34
import com.sun.tools.javac.util.*;
06bc494ca11e Initial load
duke
parents:
diff changeset
    35
import com.sun.tools.javac.util.JCDiagnostic.DiagnosticPosition;
06bc494ca11e Initial load
duke
parents:
diff changeset
    36
import com.sun.tools.javac.util.List;
06bc494ca11e Initial load
duke
parents:
diff changeset
    37
06bc494ca11e Initial load
duke
parents:
diff changeset
    38
import static com.sun.tools.javac.code.Flags.*;
27224
228abfa87080 8054457: Refactor Symbol kinds from small ints to an enum
emc
parents: 27124
diff changeset
    39
import static com.sun.tools.javac.code.Kinds.Kind.*;
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
    40
import static com.sun.tools.javac.code.Scope.LookupKind.NON_RECURSIVE;
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
    41
import static com.sun.tools.javac.code.TypeTag.CLASS;
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
    42
import static com.sun.tools.javac.code.TypeTag.TYPEVAR;
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
    43
import static com.sun.tools.javac.code.TypeTag.VOID;
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
    44
import static com.sun.tools.javac.comp.CompileStates.CompileState;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    45
06bc494ca11e Initial load
duke
parents:
diff changeset
    46
/** This pass translates Generic Java to conventional Java.
06bc494ca11e Initial load
duke
parents:
diff changeset
    47
 *
5847
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    48
 *  <p><b>This is NOT part of any supported API.
1908176fd6e3 6944312: Potential rebranding issues in openjdk/langtools repository sources
jjg
parents: 5520
diff changeset
    49
 *  If you write code that depends on this, you do so at your own risk.
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    50
 *  This code and its internal interfaces are subject to change or
06bc494ca11e Initial load
duke
parents:
diff changeset
    51
 *  deletion without notice.</b>
06bc494ca11e Initial load
duke
parents:
diff changeset
    52
 */
06bc494ca11e Initial load
duke
parents:
diff changeset
    53
public class TransTypes extends TreeTranslator {
06bc494ca11e Initial load
duke
parents:
diff changeset
    54
    /** The context key for the TransTypes phase. */
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 20249
diff changeset
    55
    protected static final Context.Key<TransTypes> transTypesKey = new Context.Key<>();
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    56
06bc494ca11e Initial load
duke
parents:
diff changeset
    57
    /** Get the instance for this context. */
06bc494ca11e Initial load
duke
parents:
diff changeset
    58
    public static TransTypes instance(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    59
        TransTypes instance = context.get(transTypesKey);
06bc494ca11e Initial load
duke
parents:
diff changeset
    60
        if (instance == null)
06bc494ca11e Initial load
duke
parents:
diff changeset
    61
            instance = new TransTypes(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    62
        return instance;
06bc494ca11e Initial load
duke
parents:
diff changeset
    63
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    64
1260
a772ba9ba43d 6574134: Allow for alternative implementation of Name Table with garbage collection of name bytes
jjg
parents: 735
diff changeset
    65
    private Names names;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    66
    private Log log;
06bc494ca11e Initial load
duke
parents:
diff changeset
    67
    private Symtab syms;
06bc494ca11e Initial load
duke
parents:
diff changeset
    68
    private TreeMaker make;
06bc494ca11e Initial load
duke
parents:
diff changeset
    69
    private Enter enter;
06bc494ca11e Initial load
duke
parents:
diff changeset
    70
    private Types types;
06bc494ca11e Initial load
duke
parents:
diff changeset
    71
    private final Resolve resolve;
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    72
    private final CompileStates compileStates;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    73
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    74
    /** Switch: is complex graph inference supported? */
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    75
    private final boolean allowGraphInference;
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    76
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    77
    /** Switch: are default methods supported? */
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    78
    private final boolean allowInterfaceBridges;
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
    79
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    80
    protected TransTypes(Context context) {
06bc494ca11e Initial load
duke
parents:
diff changeset
    81
        context.put(transTypesKey, this);
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
    82
        compileStates = CompileStates.instance(context);
1260
a772ba9ba43d 6574134: Allow for alternative implementation of Name Table with garbage collection of name bytes
jjg
parents: 735
diff changeset
    83
        names = Names.instance(context);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    84
        log = Log.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    85
        syms = Symtab.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    86
        enter = Enter.instance(context);
22163
3651128c74eb 8030244: Update langtools to use Diamond
briangoetz
parents: 20249
diff changeset
    87
        overridden = new HashMap<>();
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    88
        types = Types.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    89
        make = TreeMaker.instance(context);
06bc494ca11e Initial load
duke
parents:
diff changeset
    90
        resolve = Resolve.instance(context);
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    91
        Source source = Source.instance(context);
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    92
        allowInterfaceBridges = source.allowDefaultMethods();
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
    93
        allowGraphInference = source.allowGraphInference();
10
06bc494ca11e Initial load
duke
parents:
diff changeset
    94
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
    95
06bc494ca11e Initial load
duke
parents:
diff changeset
    96
    /** A hashtable mapping bridge methods to the methods they override after
06bc494ca11e Initial load
duke
parents:
diff changeset
    97
     *  type erasure.
06bc494ca11e Initial load
duke
parents:
diff changeset
    98
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
    99
    Map<MethodSymbol,MethodSymbol> overridden;
06bc494ca11e Initial load
duke
parents:
diff changeset
   100
06bc494ca11e Initial load
duke
parents:
diff changeset
   101
    /** Construct an attributed tree for a cast of expression to target type,
06bc494ca11e Initial load
duke
parents:
diff changeset
   102
     *  unless it already has precisely that type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   103
     *  @param tree    The expression tree.
06bc494ca11e Initial load
duke
parents:
diff changeset
   104
     *  @param target  The target type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   105
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   106
    JCExpression cast(JCExpression tree, Type target) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   107
        int oldpos = make.pos;
06bc494ca11e Initial load
duke
parents:
diff changeset
   108
        make.at(tree.pos);
06bc494ca11e Initial load
duke
parents:
diff changeset
   109
        if (!types.isSameType(tree.type, target)) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   110
            if (!resolve.isAccessible(env, target.tsym))
12082
32bdf6ef6388 7151070: NullPointerException in Resolve.isAccessible
mcimadamore
parents: 8032
diff changeset
   111
                resolve.logAccessErrorInternal(env, tree, target);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   112
            tree = make.TypeCast(make.Type(target), tree).setType(target);
06bc494ca11e Initial load
duke
parents:
diff changeset
   113
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   114
        make.pos = oldpos;
06bc494ca11e Initial load
duke
parents:
diff changeset
   115
        return tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   116
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   117
06bc494ca11e Initial load
duke
parents:
diff changeset
   118
    /** Construct an attributed tree to coerce an expression to some erased
06bc494ca11e Initial load
duke
parents:
diff changeset
   119
     *  target type, unless the expression is already assignable to that type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   120
     *  If target type is a constant type, use its base type instead.
06bc494ca11e Initial load
duke
parents:
diff changeset
   121
     *  @param tree    The expression tree.
06bc494ca11e Initial load
duke
parents:
diff changeset
   122
     *  @param target  The target type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   123
     */
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   124
    public JCExpression coerce(Env<AttrContext> env, JCExpression tree, Type target) {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   125
        Env<AttrContext> prevEnv = this.env;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   126
        try {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   127
            this.env = env;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   128
            return coerce(tree, target);
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   129
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   130
        finally {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   131
            this.env = prevEnv;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   132
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   133
    }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   134
    JCExpression coerce(JCExpression tree, Type target) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   135
        Type btarget = target.baseType();
06bc494ca11e Initial load
duke
parents:
diff changeset
   136
        if (tree.type.isPrimitive() == target.isPrimitive()) {
14547
86d8d242b0c4 8003280: Add lambda tests
mcimadamore
parents: 14365
diff changeset
   137
            return types.isAssignable(tree.type, btarget, types.noWarnings)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   138
                ? tree
06bc494ca11e Initial load
duke
parents:
diff changeset
   139
                : cast(tree, btarget);
06bc494ca11e Initial load
duke
parents:
diff changeset
   140
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   141
        return tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   142
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   143
06bc494ca11e Initial load
duke
parents:
diff changeset
   144
    /** Given an erased reference type, assume this type as the tree's type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   145
     *  Then, coerce to some given target type unless target type is null.
06bc494ca11e Initial load
duke
parents:
diff changeset
   146
     *  This operation is used in situations like the following:
06bc494ca11e Initial load
duke
parents:
diff changeset
   147
     *
14259
fb94a1df0d53 8000208: fix langtools javadoc comment issues
jjg
parents: 14258
diff changeset
   148
     *  <pre>{@code
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   149
     *  class Cell<A> { A value; }
06bc494ca11e Initial load
duke
parents:
diff changeset
   150
     *  ...
06bc494ca11e Initial load
duke
parents:
diff changeset
   151
     *  Cell<Integer> cell;
06bc494ca11e Initial load
duke
parents:
diff changeset
   152
     *  Integer x = cell.value;
14259
fb94a1df0d53 8000208: fix langtools javadoc comment issues
jjg
parents: 14258
diff changeset
   153
     *  }</pre>
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   154
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   155
     *  Since the erasure of Cell.value is Object, but the type
06bc494ca11e Initial load
duke
parents:
diff changeset
   156
     *  of cell.value in the assignment is Integer, we need to
06bc494ca11e Initial load
duke
parents:
diff changeset
   157
     *  adjust the original type of cell.value to Object, and insert
06bc494ca11e Initial load
duke
parents:
diff changeset
   158
     *  a cast to Integer. That is, the last assignment becomes:
06bc494ca11e Initial load
duke
parents:
diff changeset
   159
     *
14259
fb94a1df0d53 8000208: fix langtools javadoc comment issues
jjg
parents: 14258
diff changeset
   160
     *  <pre>{@code
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   161
     *  Integer x = (Integer)cell.value;
14259
fb94a1df0d53 8000208: fix langtools javadoc comment issues
jjg
parents: 14258
diff changeset
   162
     *  }</pre>
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   163
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   164
     *  @param tree       The expression tree whose type might need adjustment.
06bc494ca11e Initial load
duke
parents:
diff changeset
   165
     *  @param erasedType The expression's type after erasure.
06bc494ca11e Initial load
duke
parents:
diff changeset
   166
     *  @param target     The target type, which is usually the erasure of the
06bc494ca11e Initial load
duke
parents:
diff changeset
   167
     *                    expression's original type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   168
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   169
    JCExpression retype(JCExpression tree, Type erasedType, Type target) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   170
//      System.err.println("retype " + tree + " to " + erasedType);//DEBUG
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
   171
        if (!erasedType.isPrimitive()) {
17810
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   172
            if (target != null && target.isPrimitive()) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   173
                target = erasure(tree.type);
17810
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   174
            }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   175
            tree.type = erasedType;
17810
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   176
            if (target != null) {
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   177
                return coerce(tree, target);
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   178
            }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   179
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   180
        return tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   181
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   182
06bc494ca11e Initial load
duke
parents:
diff changeset
   183
    /** Translate method argument list, casting each argument
06bc494ca11e Initial load
duke
parents:
diff changeset
   184
     *  to its corresponding type in a list of target types.
06bc494ca11e Initial load
duke
parents:
diff changeset
   185
     *  @param _args            The method argument list.
06bc494ca11e Initial load
duke
parents:
diff changeset
   186
     *  @param parameters       The list of target types.
06bc494ca11e Initial load
duke
parents:
diff changeset
   187
     *  @param varargsElement   The erasure of the varargs element type,
06bc494ca11e Initial load
duke
parents:
diff changeset
   188
     *  or null if translating a non-varargs invocation
06bc494ca11e Initial load
duke
parents:
diff changeset
   189
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   190
    <T extends JCTree> List<T> translateArgs(List<T> _args,
06bc494ca11e Initial load
duke
parents:
diff changeset
   191
                                           List<Type> parameters,
06bc494ca11e Initial load
duke
parents:
diff changeset
   192
                                           Type varargsElement) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   193
        if (parameters.isEmpty()) return _args;
06bc494ca11e Initial load
duke
parents:
diff changeset
   194
        List<T> args = _args;
06bc494ca11e Initial load
duke
parents:
diff changeset
   195
        while (parameters.tail.nonEmpty()) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   196
            args.head = translate(args.head, parameters.head);
06bc494ca11e Initial load
duke
parents:
diff changeset
   197
            args = args.tail;
06bc494ca11e Initial load
duke
parents:
diff changeset
   198
            parameters = parameters.tail;
06bc494ca11e Initial load
duke
parents:
diff changeset
   199
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   200
        Type parameter = parameters.head;
8032
e1aa25ccdabb 6396503: javac should not require assertions enabled
jjg
parents: 8031
diff changeset
   201
        Assert.check(varargsElement != null || args.length() == 1);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   202
        if (varargsElement != null) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   203
            while (args.nonEmpty()) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   204
                args.head = translate(args.head, varargsElement);
06bc494ca11e Initial load
duke
parents:
diff changeset
   205
                args = args.tail;
06bc494ca11e Initial load
duke
parents:
diff changeset
   206
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   207
        } else {
06bc494ca11e Initial load
duke
parents:
diff changeset
   208
            args.head = translate(args.head, parameter);
06bc494ca11e Initial load
duke
parents:
diff changeset
   209
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   210
        return _args;
06bc494ca11e Initial load
duke
parents:
diff changeset
   211
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   212
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   213
    public <T extends JCTree> List<T> translateArgs(List<T> _args,
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   214
                                           List<Type> parameters,
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   215
                                           Type varargsElement,
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   216
                                           Env<AttrContext> localEnv) {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   217
        Env<AttrContext> prevEnv = env;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   218
        try {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   219
            env = localEnv;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   220
            return translateArgs(_args, parameters, varargsElement);
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   221
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   222
        finally {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   223
            env = prevEnv;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   224
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   225
    }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   226
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   227
    /** Add a bridge definition and enter corresponding method symbol in
06bc494ca11e Initial load
duke
parents:
diff changeset
   228
     *  local scope of origin.
06bc494ca11e Initial load
duke
parents:
diff changeset
   229
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   230
     *  @param pos     The source code position to be used for the definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   231
     *  @param meth    The method for which a bridge needs to be added
06bc494ca11e Initial load
duke
parents:
diff changeset
   232
     *  @param impl    That method's implementation (possibly the method itself)
06bc494ca11e Initial load
duke
parents:
diff changeset
   233
     *  @param origin  The class to which the bridge will be added
06bc494ca11e Initial load
duke
parents:
diff changeset
   234
     *  @param hypothetical
06bc494ca11e Initial load
duke
parents:
diff changeset
   235
     *                 True if the bridge method is not strictly necessary in the
06bc494ca11e Initial load
duke
parents:
diff changeset
   236
     *                 binary, but is represented in the symbol table to detect
06bc494ca11e Initial load
duke
parents:
diff changeset
   237
     *                 erasure clashes.
06bc494ca11e Initial load
duke
parents:
diff changeset
   238
     *  @param bridges The list buffer to which the bridge will be added
06bc494ca11e Initial load
duke
parents:
diff changeset
   239
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   240
    void addBridge(DiagnosticPosition pos,
06bc494ca11e Initial load
duke
parents:
diff changeset
   241
                   MethodSymbol meth,
06bc494ca11e Initial load
duke
parents:
diff changeset
   242
                   MethodSymbol impl,
06bc494ca11e Initial load
duke
parents:
diff changeset
   243
                   ClassSymbol origin,
06bc494ca11e Initial load
duke
parents:
diff changeset
   244
                   boolean hypothetical,
06bc494ca11e Initial load
duke
parents:
diff changeset
   245
                   ListBuffer<JCTree> bridges) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   246
        make.at(pos);
06bc494ca11e Initial load
duke
parents:
diff changeset
   247
        Type origType = types.memberType(origin.type, meth);
06bc494ca11e Initial load
duke
parents:
diff changeset
   248
        Type origErasure = erasure(origType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   249
06bc494ca11e Initial load
duke
parents:
diff changeset
   250
        // Create a bridge method symbol and a bridge definition without a body.
06bc494ca11e Initial load
duke
parents:
diff changeset
   251
        Type bridgeType = meth.erasure(types);
24793
bb58a9f1ae10 8044487: Fix for 8042785 causes regression tests to fail with java.lang.VerifyError
vromero
parents: 24792
diff changeset
   252
        long flags = impl.flags() & AccessFlags | SYNTHETIC | BRIDGE |
18730
95354d510139 8016281: The SAM method should be passed to the metafactory as a MethodType not a MethodHandle
mcimadamore
parents: 18412
diff changeset
   253
                (origin.isInterface() ? DEFAULT : 0);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   254
        if (hypothetical) flags |= HYPOTHETICAL;
06bc494ca11e Initial load
duke
parents:
diff changeset
   255
        MethodSymbol bridge = new MethodSymbol(flags,
06bc494ca11e Initial load
duke
parents:
diff changeset
   256
                                               meth.name,
06bc494ca11e Initial load
duke
parents:
diff changeset
   257
                                               bridgeType,
06bc494ca11e Initial load
duke
parents:
diff changeset
   258
                                               origin);
18000
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   259
        /* once JDK-6996415 is solved it should be checked if this approach can
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   260
         * be applied to method addOverrideBridgesIfNeeded
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   261
         */
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   262
        bridge.params = createBridgeParams(impl, bridge, bridgeType);
18010
604faee85350 8004643: Reduce javac space overhead introduced with compiler support for repeating annotations
jjg
parents: 18000
diff changeset
   263
        bridge.setAttributes(impl);
18000
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   264
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   265
        if (!hypothetical) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   266
            JCMethodDecl md = make.MethodDef(bridge, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   267
06bc494ca11e Initial load
duke
parents:
diff changeset
   268
            // The bridge calls this.impl(..), if we have an implementation
06bc494ca11e Initial load
duke
parents:
diff changeset
   269
            // in the current class, super.impl(...) otherwise.
06bc494ca11e Initial load
duke
parents:
diff changeset
   270
            JCExpression receiver = (impl.owner == origin)
06bc494ca11e Initial load
duke
parents:
diff changeset
   271
                ? make.This(origin.erasure(types))
06bc494ca11e Initial load
duke
parents:
diff changeset
   272
                : make.Super(types.supertype(origin.type).tsym.erasure(types), origin);
06bc494ca11e Initial load
duke
parents:
diff changeset
   273
06bc494ca11e Initial load
duke
parents:
diff changeset
   274
            // The type returned from the original method.
06bc494ca11e Initial load
duke
parents:
diff changeset
   275
            Type calltype = erasure(impl.type.getReturnType());
06bc494ca11e Initial load
duke
parents:
diff changeset
   276
06bc494ca11e Initial load
duke
parents:
diff changeset
   277
            // Construct a call of  this.impl(params), or super.impl(params),
06bc494ca11e Initial load
duke
parents:
diff changeset
   278
            // casting params and possibly results as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   279
            JCExpression call =
06bc494ca11e Initial load
duke
parents:
diff changeset
   280
                make.Apply(
06bc494ca11e Initial load
duke
parents:
diff changeset
   281
                           null,
06bc494ca11e Initial load
duke
parents:
diff changeset
   282
                           make.Select(receiver, impl).setType(calltype),
06bc494ca11e Initial load
duke
parents:
diff changeset
   283
                           translateArgs(make.Idents(md.params), origErasure.getParameterTypes(), null))
06bc494ca11e Initial load
duke
parents:
diff changeset
   284
                .setType(calltype);
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
   285
            JCStatement stat = (origErasure.getReturnType().hasTag(VOID))
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   286
                ? make.Exec(call)
06bc494ca11e Initial load
duke
parents:
diff changeset
   287
                : make.Return(coerce(call, bridgeType.getReturnType()));
06bc494ca11e Initial load
duke
parents:
diff changeset
   288
            md.body = make.Block(0, List.of(stat));
06bc494ca11e Initial load
duke
parents:
diff changeset
   289
06bc494ca11e Initial load
duke
parents:
diff changeset
   290
            // Add bridge to `bridges' buffer
06bc494ca11e Initial load
duke
parents:
diff changeset
   291
            bridges.append(md);
06bc494ca11e Initial load
duke
parents:
diff changeset
   292
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   293
06bc494ca11e Initial load
duke
parents:
diff changeset
   294
        // Add bridge to scope of enclosing class and `overridden' table.
06bc494ca11e Initial load
duke
parents:
diff changeset
   295
        origin.members().enter(bridge);
06bc494ca11e Initial load
duke
parents:
diff changeset
   296
        overridden.put(bridge, meth);
06bc494ca11e Initial load
duke
parents:
diff changeset
   297
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   298
18000
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   299
    private List<VarSymbol> createBridgeParams(MethodSymbol impl, MethodSymbol bridge,
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   300
            Type bridgeType) {
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   301
        List<VarSymbol> bridgeParams = null;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   302
        if (impl.params != null) {
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   303
            bridgeParams = List.nil();
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   304
            List<VarSymbol> implParams = impl.params;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   305
            Type.MethodType mType = (Type.MethodType)bridgeType;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   306
            List<Type> argTypes = mType.argtypes;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   307
            while (implParams.nonEmpty() && argTypes.nonEmpty()) {
19941
8b91e8eb2d20 7047734: javac, the LVT is not generated correctly in several scenarios
vromero
parents: 19939
diff changeset
   308
                VarSymbol param = new VarSymbol(implParams.head.flags() | SYNTHETIC | PARAMETER,
18000
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   309
                        implParams.head.name, argTypes.head, bridge);
18010
604faee85350 8004643: Reduce javac space overhead introduced with compiler support for repeating annotations
jjg
parents: 18000
diff changeset
   310
                param.setAttributes(implParams.head);
18000
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   311
                bridgeParams = bridgeParams.append(param);
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   312
                implParams = implParams.tail;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   313
                argTypes = argTypes.tail;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   314
            }
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   315
        }
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   316
        return bridgeParams;
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   317
    }
5d29ce00a7a2 6695379: Copy method annotations and parameter annotations to synthetic bridge methods
vromero
parents: 17810
diff changeset
   318
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   319
    /** Add bridge if given symbol is a non-private, non-static member
06bc494ca11e Initial load
duke
parents:
diff changeset
   320
     *  of the given class, which is either defined in the class or non-final
06bc494ca11e Initial load
duke
parents:
diff changeset
   321
     *  inherited, and one of the two following conditions holds:
06bc494ca11e Initial load
duke
parents:
diff changeset
   322
     *  1. The method's type changes in the given class, as compared to the
06bc494ca11e Initial load
duke
parents:
diff changeset
   323
     *     class where the symbol was defined, (in this case
06bc494ca11e Initial load
duke
parents:
diff changeset
   324
     *     we have extended a parameterized class with non-trivial parameters).
06bc494ca11e Initial load
duke
parents:
diff changeset
   325
     *  2. The method has an implementation with a different erased return type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   326
     *     (in this case we have used co-variant returns).
06bc494ca11e Initial load
duke
parents:
diff changeset
   327
     *  If a bridge already exists in some other class, no new bridge is added.
06bc494ca11e Initial load
duke
parents:
diff changeset
   328
     *  Instead, it is checked that the bridge symbol overrides the method symbol.
06bc494ca11e Initial load
duke
parents:
diff changeset
   329
     *  (Spec ???).
06bc494ca11e Initial load
duke
parents:
diff changeset
   330
     *  todo: what about bridges for privates???
06bc494ca11e Initial load
duke
parents:
diff changeset
   331
     *
06bc494ca11e Initial load
duke
parents:
diff changeset
   332
     *  @param pos     The source code position to be used for the definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   333
     *  @param sym     The symbol for which a bridge might have to be added.
06bc494ca11e Initial load
duke
parents:
diff changeset
   334
     *  @param origin  The class in which the bridge would go.
06bc494ca11e Initial load
duke
parents:
diff changeset
   335
     *  @param bridges The list buffer to which the bridge would be added.
06bc494ca11e Initial load
duke
parents:
diff changeset
   336
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   337
    void addBridgeIfNeeded(DiagnosticPosition pos,
06bc494ca11e Initial load
duke
parents:
diff changeset
   338
                           Symbol sym,
06bc494ca11e Initial load
duke
parents:
diff changeset
   339
                           ClassSymbol origin,
06bc494ca11e Initial load
duke
parents:
diff changeset
   340
                           ListBuffer<JCTree> bridges) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   341
        if (sym.kind == MTH &&
06bc494ca11e Initial load
duke
parents:
diff changeset
   342
            sym.name != names.init &&
6591
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   343
            (sym.flags() & (PRIVATE | STATIC)) == 0 &&
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   344
            (sym.flags() & (SYNTHETIC | OVERRIDE_BRIDGE)) != SYNTHETIC &&
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   345
            sym.isMemberOf(origin, types))
06bc494ca11e Initial load
duke
parents:
diff changeset
   346
        {
06bc494ca11e Initial load
duke
parents:
diff changeset
   347
            MethodSymbol meth = (MethodSymbol)sym;
06bc494ca11e Initial load
duke
parents:
diff changeset
   348
            MethodSymbol bridge = meth.binaryImplementation(origin, types);
6591
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   349
            MethodSymbol impl = meth.implementation(origin, types, true, overrideBridgeFilter);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   350
            if (bridge == null ||
06bc494ca11e Initial load
duke
parents:
diff changeset
   351
                bridge == meth ||
06bc494ca11e Initial load
duke
parents:
diff changeset
   352
                (impl != null && !bridge.owner.isSubClass(impl.owner, types))) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   353
                // No bridge was added yet.
06bc494ca11e Initial load
duke
parents:
diff changeset
   354
                if (impl != null && isBridgeNeeded(meth, impl, origin.type)) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   355
                    addBridge(pos, meth, impl, origin, bridge==impl, bridges);
06bc494ca11e Initial load
duke
parents:
diff changeset
   356
                } else if (impl == meth
06bc494ca11e Initial load
duke
parents:
diff changeset
   357
                           && impl.owner != origin
06bc494ca11e Initial load
duke
parents:
diff changeset
   358
                           && (impl.flags() & FINAL) == 0
06bc494ca11e Initial load
duke
parents:
diff changeset
   359
                           && (meth.flags() & (ABSTRACT|PUBLIC)) == PUBLIC
06bc494ca11e Initial load
duke
parents:
diff changeset
   360
                           && (origin.flags() & PUBLIC) > (impl.owner.flags() & PUBLIC)) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   361
                    // this is to work around a horrible but permanent
06bc494ca11e Initial load
duke
parents:
diff changeset
   362
                    // reflection design error.
06bc494ca11e Initial load
duke
parents:
diff changeset
   363
                    addBridge(pos, meth, impl, origin, false, bridges);
06bc494ca11e Initial load
duke
parents:
diff changeset
   364
                }
6591
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   365
            } else if ((bridge.flags() & (SYNTHETIC | OVERRIDE_BRIDGE)) == SYNTHETIC) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   366
                MethodSymbol other = overridden.get(bridge);
06bc494ca11e Initial load
duke
parents:
diff changeset
   367
                if (other != null && other != meth) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   368
                    if (impl == null || !impl.overrides(other, origin, types, true)) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   369
                        // Bridge for other symbol pair was added
06bc494ca11e Initial load
duke
parents:
diff changeset
   370
                        log.error(pos, "name.clash.same.erasure.no.override",
06bc494ca11e Initial load
duke
parents:
diff changeset
   371
                                  other, other.location(origin.type, types),
06bc494ca11e Initial load
duke
parents:
diff changeset
   372
                                  meth,  meth.location(origin.type, types));
06bc494ca11e Initial load
duke
parents:
diff changeset
   373
                    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   374
                }
06bc494ca11e Initial load
duke
parents:
diff changeset
   375
            } else if (!bridge.overrides(meth, origin, types, true)) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   376
                // Accidental binary override without source override.
06bc494ca11e Initial load
duke
parents:
diff changeset
   377
                if (bridge.owner == origin ||
06bc494ca11e Initial load
duke
parents:
diff changeset
   378
                    types.asSuper(bridge.owner.type, meth.owner) == null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   379
                    // Don't diagnose the problem if it would already
06bc494ca11e Initial load
duke
parents:
diff changeset
   380
                    // have been reported in the superclass
06bc494ca11e Initial load
duke
parents:
diff changeset
   381
                    log.error(pos, "name.clash.same.erasure.no.override",
06bc494ca11e Initial load
duke
parents:
diff changeset
   382
                              bridge, bridge.location(origin.type, types),
06bc494ca11e Initial load
duke
parents:
diff changeset
   383
                              meth,  meth.location(origin.type, types));
06bc494ca11e Initial load
duke
parents:
diff changeset
   384
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   385
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   386
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   387
    // where
18730
95354d510139 8016281: The SAM method should be passed to the metafactory as a MethodType not a MethodHandle
mcimadamore
parents: 18412
diff changeset
   388
        private Filter<Symbol> overrideBridgeFilter = new Filter<Symbol>() {
6591
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   389
            public boolean accepts(Symbol s) {
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   390
                return (s.flags() & (SYNTHETIC | OVERRIDE_BRIDGE)) != SYNTHETIC;
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   391
            }
a953c8c6b85e 6337171: javac should create bridge methods when type variable bounds restricted
mcimadamore
parents: 6148
diff changeset
   392
        };
18730
95354d510139 8016281: The SAM method should be passed to the metafactory as a MethodType not a MethodHandle
mcimadamore
parents: 18412
diff changeset
   393
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   394
        /**
06bc494ca11e Initial load
duke
parents:
diff changeset
   395
         * @param method The symbol for which a bridge might have to be added
06bc494ca11e Initial load
duke
parents:
diff changeset
   396
         * @param impl The implementation of method
06bc494ca11e Initial load
duke
parents:
diff changeset
   397
         * @param dest The type in which the bridge would go
06bc494ca11e Initial load
duke
parents:
diff changeset
   398
         */
06bc494ca11e Initial load
duke
parents:
diff changeset
   399
        private boolean isBridgeNeeded(MethodSymbol method,
06bc494ca11e Initial load
duke
parents:
diff changeset
   400
                                       MethodSymbol impl,
06bc494ca11e Initial load
duke
parents:
diff changeset
   401
                                       Type dest) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   402
            if (impl != method) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   403
                // If either method or impl have different erasures as
06bc494ca11e Initial load
duke
parents:
diff changeset
   404
                // members of dest, a bridge is needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   405
                Type method_erasure = method.erasure(types);
06bc494ca11e Initial load
duke
parents:
diff changeset
   406
                if (!isSameMemberWhenErased(dest, method, method_erasure))
06bc494ca11e Initial load
duke
parents:
diff changeset
   407
                    return true;
06bc494ca11e Initial load
duke
parents:
diff changeset
   408
                Type impl_erasure = impl.erasure(types);
06bc494ca11e Initial load
duke
parents:
diff changeset
   409
                if (!isSameMemberWhenErased(dest, impl, impl_erasure))
06bc494ca11e Initial load
duke
parents:
diff changeset
   410
                    return true;
06bc494ca11e Initial load
duke
parents:
diff changeset
   411
06bc494ca11e Initial load
duke
parents:
diff changeset
   412
                // If the erasure of the return type is different, a
06bc494ca11e Initial load
duke
parents:
diff changeset
   413
                // bridge is needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   414
                return !types.isSameType(impl_erasure.getReturnType(),
06bc494ca11e Initial load
duke
parents:
diff changeset
   415
                                         method_erasure.getReturnType());
06bc494ca11e Initial load
duke
parents:
diff changeset
   416
            } else {
06bc494ca11e Initial load
duke
parents:
diff changeset
   417
               // method and impl are the same...
06bc494ca11e Initial load
duke
parents:
diff changeset
   418
                if ((method.flags() & ABSTRACT) != 0) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   419
                    // ...and abstract so a bridge is not needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   420
                    // Concrete subclasses will bridge as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   421
                    return false;
06bc494ca11e Initial load
duke
parents:
diff changeset
   422
                }
06bc494ca11e Initial load
duke
parents:
diff changeset
   423
06bc494ca11e Initial load
duke
parents:
diff changeset
   424
                // The erasure of the return type is always the same
06bc494ca11e Initial load
duke
parents:
diff changeset
   425
                // for the same symbol.  Reducing the three tests in
06bc494ca11e Initial load
duke
parents:
diff changeset
   426
                // the other branch to just one:
06bc494ca11e Initial load
duke
parents:
diff changeset
   427
                return !isSameMemberWhenErased(dest, method, method.erasure(types));
06bc494ca11e Initial load
duke
parents:
diff changeset
   428
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   429
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   430
        /**
06bc494ca11e Initial load
duke
parents:
diff changeset
   431
         * Lookup the method as a member of the type.  Compare the
06bc494ca11e Initial load
duke
parents:
diff changeset
   432
         * erasures.
06bc494ca11e Initial load
duke
parents:
diff changeset
   433
         * @param type the class where to look for the method
06bc494ca11e Initial load
duke
parents:
diff changeset
   434
         * @param method the method to look for in class
06bc494ca11e Initial load
duke
parents:
diff changeset
   435
         * @param erasure the erasure of method
06bc494ca11e Initial load
duke
parents:
diff changeset
   436
         */
06bc494ca11e Initial load
duke
parents:
diff changeset
   437
        private boolean isSameMemberWhenErased(Type type,
06bc494ca11e Initial load
duke
parents:
diff changeset
   438
                                               MethodSymbol method,
06bc494ca11e Initial load
duke
parents:
diff changeset
   439
                                               Type erasure) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   440
            return types.isSameType(erasure(types.memberType(type, method)),
06bc494ca11e Initial load
duke
parents:
diff changeset
   441
                                    erasure);
06bc494ca11e Initial load
duke
parents:
diff changeset
   442
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   443
06bc494ca11e Initial load
duke
parents:
diff changeset
   444
    void addBridges(DiagnosticPosition pos,
06bc494ca11e Initial load
duke
parents:
diff changeset
   445
                    TypeSymbol i,
06bc494ca11e Initial load
duke
parents:
diff changeset
   446
                    ClassSymbol origin,
06bc494ca11e Initial load
duke
parents:
diff changeset
   447
                    ListBuffer<JCTree> bridges) {
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   448
        for (Symbol sym : i.members().getSymbols(NON_RECURSIVE))
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   449
            addBridgeIfNeeded(pos, sym, origin, bridges);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   450
        for (List<Type> l = types.interfaces(i.type); l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   451
            addBridges(pos, l.head.tsym, origin, bridges);
06bc494ca11e Initial load
duke
parents:
diff changeset
   452
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   453
06bc494ca11e Initial load
duke
parents:
diff changeset
   454
    /** Add all necessary bridges to some class appending them to list buffer.
06bc494ca11e Initial load
duke
parents:
diff changeset
   455
     *  @param pos     The source code position to be used for the bridges.
06bc494ca11e Initial load
duke
parents:
diff changeset
   456
     *  @param origin  The class in which the bridges go.
06bc494ca11e Initial load
duke
parents:
diff changeset
   457
     *  @param bridges The list buffer to which the bridges are added.
06bc494ca11e Initial load
duke
parents:
diff changeset
   458
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   459
    void addBridges(DiagnosticPosition pos, ClassSymbol origin, ListBuffer<JCTree> bridges) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   460
        Type st = types.supertype(origin.type);
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
   461
        while (st.hasTag(CLASS)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   462
//          if (isSpecialization(st))
06bc494ca11e Initial load
duke
parents:
diff changeset
   463
            addBridges(pos, st.tsym, origin, bridges);
06bc494ca11e Initial load
duke
parents:
diff changeset
   464
            st = types.supertype(st);
06bc494ca11e Initial load
duke
parents:
diff changeset
   465
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   466
        for (List<Type> l = types.interfaces(origin.type); l.nonEmpty(); l = l.tail)
06bc494ca11e Initial load
duke
parents:
diff changeset
   467
//          if (isSpecialization(l.head))
06bc494ca11e Initial load
duke
parents:
diff changeset
   468
            addBridges(pos, l.head.tsym, origin, bridges);
06bc494ca11e Initial load
duke
parents:
diff changeset
   469
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   470
06bc494ca11e Initial load
duke
parents:
diff changeset
   471
/* ************************************************************************
06bc494ca11e Initial load
duke
parents:
diff changeset
   472
 * Visitor methods
06bc494ca11e Initial load
duke
parents:
diff changeset
   473
 *************************************************************************/
06bc494ca11e Initial load
duke
parents:
diff changeset
   474
06bc494ca11e Initial load
duke
parents:
diff changeset
   475
    /** Visitor argument: proto-type.
06bc494ca11e Initial load
duke
parents:
diff changeset
   476
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   477
    private Type pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   478
06bc494ca11e Initial load
duke
parents:
diff changeset
   479
    /** Visitor method: perform a type translation on tree.
06bc494ca11e Initial load
duke
parents:
diff changeset
   480
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   481
    public <T extends JCTree> T translate(T tree, Type pt) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   482
        Type prevPt = this.pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   483
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   484
            this.pt = pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   485
            return translate(tree);
06bc494ca11e Initial load
duke
parents:
diff changeset
   486
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   487
            this.pt = prevPt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   488
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   489
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   490
06bc494ca11e Initial load
duke
parents:
diff changeset
   491
    /** Visitor method: perform a type translation on list of trees.
06bc494ca11e Initial load
duke
parents:
diff changeset
   492
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   493
    public <T extends JCTree> List<T> translate(List<T> trees, Type pt) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   494
        Type prevPt = this.pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   495
        List<T> res;
06bc494ca11e Initial load
duke
parents:
diff changeset
   496
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   497
            this.pt = pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   498
            res = translate(trees);
06bc494ca11e Initial load
duke
parents:
diff changeset
   499
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   500
            this.pt = prevPt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   501
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   502
        return res;
06bc494ca11e Initial load
duke
parents:
diff changeset
   503
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   504
06bc494ca11e Initial load
duke
parents:
diff changeset
   505
    public void visitClassDef(JCClassDecl tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   506
        translateClass(tree.sym);
06bc494ca11e Initial load
duke
parents:
diff changeset
   507
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   508
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   509
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   510
    JCTree currentMethod = null;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   511
    public void visitMethodDef(JCMethodDecl tree) {
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   512
        JCTree previousMethod = currentMethod;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   513
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   514
            currentMethod = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   515
            tree.restype = translate(tree.restype, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   516
            tree.typarams = List.nil();
06bc494ca11e Initial load
duke
parents:
diff changeset
   517
            tree.params = translateVarDefs(tree.params);
15385
ee1eebe7e210 8006775: JSR 308: Compiler changes in JDK8
jjg
parents: 15374
diff changeset
   518
            tree.recvparam = translate(tree.recvparam, null);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   519
            tree.thrown = translate(tree.thrown, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   520
            tree.body = translate(tree.body, tree.sym.erasure(types).getReturnType());
06bc494ca11e Initial load
duke
parents:
diff changeset
   521
            tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   522
            result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   523
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   524
            currentMethod = previousMethod;
06bc494ca11e Initial load
duke
parents:
diff changeset
   525
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   526
06bc494ca11e Initial load
duke
parents:
diff changeset
   527
        // Check that we do not introduce a name clash by erasing types.
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   528
        for (Symbol sym : tree.sym.owner.members().getSymbolsByName(tree.name)) {
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   529
            if (sym != tree.sym &&
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   530
                types.isSameType(erasure(sym.type), tree.type)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   531
                log.error(tree.pos(),
06bc494ca11e Initial load
duke
parents:
diff changeset
   532
                          "name.clash.same.erasure", tree.sym,
25443
9187d77f2c64 8031569: Refactor javac scope implementation to enable lazy imports
jlahoda
parents: 25306
diff changeset
   533
                          sym);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   534
                return;
06bc494ca11e Initial load
duke
parents:
diff changeset
   535
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   536
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   537
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   538
06bc494ca11e Initial load
duke
parents:
diff changeset
   539
    public void visitVarDef(JCVariableDecl tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   540
        tree.vartype = translate(tree.vartype, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   541
        tree.init = translate(tree.init, tree.sym.erasure(types));
06bc494ca11e Initial load
duke
parents:
diff changeset
   542
        tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   543
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   544
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   545
06bc494ca11e Initial load
duke
parents:
diff changeset
   546
    public void visitDoLoop(JCDoWhileLoop tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   547
        tree.body = translate(tree.body);
06bc494ca11e Initial load
duke
parents:
diff changeset
   548
        tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   549
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   550
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   551
06bc494ca11e Initial load
duke
parents:
diff changeset
   552
    public void visitWhileLoop(JCWhileLoop tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   553
        tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   554
        tree.body = translate(tree.body);
06bc494ca11e Initial load
duke
parents:
diff changeset
   555
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   556
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   557
06bc494ca11e Initial load
duke
parents:
diff changeset
   558
    public void visitForLoop(JCForLoop tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   559
        tree.init = translate(tree.init, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   560
        if (tree.cond != null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   561
            tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   562
        tree.step = translate(tree.step, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   563
        tree.body = translate(tree.body);
06bc494ca11e Initial load
duke
parents:
diff changeset
   564
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   565
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   566
06bc494ca11e Initial load
duke
parents:
diff changeset
   567
    public void visitForeachLoop(JCEnhancedForLoop tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   568
        tree.var = translate(tree.var, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   569
        Type iterableType = tree.expr.type;
06bc494ca11e Initial load
duke
parents:
diff changeset
   570
        tree.expr = translate(tree.expr, erasure(tree.expr.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   571
        if (types.elemtype(tree.expr.type) == null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   572
            tree.expr.type = iterableType; // preserve type for Lower
06bc494ca11e Initial load
duke
parents:
diff changeset
   573
        tree.body = translate(tree.body);
06bc494ca11e Initial load
duke
parents:
diff changeset
   574
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   575
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   576
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   577
    public void visitLambda(JCLambda tree) {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   578
        JCTree prevMethod = currentMethod;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   579
        try {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   580
            currentMethod = null;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   581
            tree.params = translate(tree.params);
22168
a48fe7ec473d 8030626: java.lang.VerifyError: Bad return type when lambda's body is in parentheses
rfield
parents: 22163
diff changeset
   582
            tree.body = translate(tree.body, tree.body.type==null? null : erasure(tree.body.type));
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   583
            tree.type = erasure(tree.type);
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   584
            result = tree;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   585
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   586
        finally {
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   587
            currentMethod = prevMethod;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   588
        }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   589
    }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   590
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   591
    public void visitSwitch(JCSwitch tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   592
        Type selsuper = types.supertype(tree.selector.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   593
        boolean enumSwitch = selsuper != null &&
06bc494ca11e Initial load
duke
parents:
diff changeset
   594
            selsuper.tsym == syms.enumSym;
06bc494ca11e Initial load
duke
parents:
diff changeset
   595
        Type target = enumSwitch ? erasure(tree.selector.type) : syms.intType;
06bc494ca11e Initial load
duke
parents:
diff changeset
   596
        tree.selector = translate(tree.selector, target);
06bc494ca11e Initial load
duke
parents:
diff changeset
   597
        tree.cases = translateCases(tree.cases);
06bc494ca11e Initial load
duke
parents:
diff changeset
   598
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   599
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   600
06bc494ca11e Initial load
duke
parents:
diff changeset
   601
    public void visitCase(JCCase tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   602
        tree.pat = translate(tree.pat, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   603
        tree.stats = translate(tree.stats);
06bc494ca11e Initial load
duke
parents:
diff changeset
   604
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   605
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   606
06bc494ca11e Initial load
duke
parents:
diff changeset
   607
    public void visitSynchronized(JCSynchronized tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   608
        tree.lock = translate(tree.lock, erasure(tree.lock.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   609
        tree.body = translate(tree.body);
06bc494ca11e Initial load
duke
parents:
diff changeset
   610
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   611
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   612
6148
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   613
    public void visitTry(JCTry tree) {
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   614
        tree.resources = translate(tree.resources, syms.autoCloseableType);
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   615
        tree.body = translate(tree.body);
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   616
        tree.catchers = translateCatchers(tree.catchers);
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   617
        tree.finalizer = translate(tree.finalizer);
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   618
        result = tree;
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   619
    }
3a8158299c51 6911256: Project Coin: Support Automatic Resource Management (ARM) blocks in the compiler
darcy
parents: 5847
diff changeset
   620
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   621
    public void visitConditional(JCConditional tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   622
        tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   623
        tree.truepart = translate(tree.truepart, erasure(tree.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   624
        tree.falsepart = translate(tree.falsepart, erasure(tree.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   625
        tree.type = erasure(tree.type);
1356
9f40f16e543b 6500343: compiler generates bad code when translating conditional expressions
mcimadamore
parents: 1260
diff changeset
   626
        result = retype(tree, tree.type, pt);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   627
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   628
06bc494ca11e Initial load
duke
parents:
diff changeset
   629
   public void visitIf(JCIf tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   630
        tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   631
        tree.thenpart = translate(tree.thenpart);
06bc494ca11e Initial load
duke
parents:
diff changeset
   632
        tree.elsepart = translate(tree.elsepart);
06bc494ca11e Initial load
duke
parents:
diff changeset
   633
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   634
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   635
06bc494ca11e Initial load
duke
parents:
diff changeset
   636
    public void visitExec(JCExpressionStatement tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   637
        tree.expr = translate(tree.expr, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   638
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   639
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   640
06bc494ca11e Initial load
duke
parents:
diff changeset
   641
    public void visitReturn(JCReturn tree) {
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   642
        tree.expr = translate(tree.expr, currentMethod != null ? types.erasure(currentMethod.type).getReturnType() : null);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   643
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   644
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   645
06bc494ca11e Initial load
duke
parents:
diff changeset
   646
    public void visitThrow(JCThrow tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   647
        tree.expr = translate(tree.expr, erasure(tree.expr.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   648
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   649
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   650
06bc494ca11e Initial load
duke
parents:
diff changeset
   651
    public void visitAssert(JCAssert tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   652
        tree.cond = translate(tree.cond, syms.booleanType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   653
        if (tree.detail != null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   654
            tree.detail = translate(tree.detail, erasure(tree.detail.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   655
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   656
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   657
06bc494ca11e Initial load
duke
parents:
diff changeset
   658
    public void visitApply(JCMethodInvocation tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   659
        tree.meth = translate(tree.meth, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   660
        Symbol meth = TreeInfo.symbol(tree.meth);
06bc494ca11e Initial load
duke
parents:
diff changeset
   661
        Type mt = meth.erasure(types);
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   662
        boolean useInstantiatedPtArgs =
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   663
                allowGraphInference && !types.isSignaturePolymorphic((MethodSymbol)meth.baseSymbol());
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   664
        List<Type> argtypes = useInstantiatedPtArgs ?
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   665
                tree.meth.type.getParameterTypes() :
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   666
                mt.getParameterTypes();
25445
603f0c93d5c9 8011044: Remove support for 1.5 and earlier source and target options
ntoda
parents: 25443
diff changeset
   667
        if (meth.name == names.init && meth.owner == syms.enumSym)
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   668
            argtypes = argtypes.tail.tail;
06bc494ca11e Initial load
duke
parents:
diff changeset
   669
        if (tree.varargsElement != null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   670
            tree.varargsElement = types.erasure(tree.varargsElement);
06bc494ca11e Initial load
duke
parents:
diff changeset
   671
        else
19133
aa085efaaa2e 8013179: assertion failure in javac when compiling with -source 1.6 -target 1.6
vromero
parents: 18730
diff changeset
   672
            if (tree.args.length() != argtypes.length()) {
aa085efaaa2e 8013179: assertion failure in javac when compiling with -source 1.6 -target 1.6
vromero
parents: 18730
diff changeset
   673
                log.error(tree.pos(),
aa085efaaa2e 8013179: assertion failure in javac when compiling with -source 1.6 -target 1.6
vromero
parents: 18730
diff changeset
   674
                              "method.invoked.with.incorrect.number.arguments",
aa085efaaa2e 8013179: assertion failure in javac when compiling with -source 1.6 -target 1.6
vromero
parents: 18730
diff changeset
   675
                              tree.args.length(), argtypes.length());
aa085efaaa2e 8013179: assertion failure in javac when compiling with -source 1.6 -target 1.6
vromero
parents: 18730
diff changeset
   676
            }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   677
        tree.args = translateArgs(tree.args, argtypes, tree.varargsElement);
06bc494ca11e Initial load
duke
parents:
diff changeset
   678
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   679
        tree.type = types.erasure(tree.type);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   680
        // Insert casts of method invocation results as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   681
        result = retype(tree, mt.getReturnType(), pt);
06bc494ca11e Initial load
duke
parents:
diff changeset
   682
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   683
06bc494ca11e Initial load
duke
parents:
diff changeset
   684
    public void visitNewClass(JCNewClass tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   685
        if (tree.encl != null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   686
            tree.encl = translate(tree.encl, erasure(tree.encl.type));
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   687
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   688
        Type erasedConstructorType = tree.constructorType != null ?
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   689
                erasure(tree.constructorType) :
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   690
                null;
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   691
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   692
        List<Type> argtypes = erasedConstructorType != null && allowGraphInference ?
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   693
                erasedConstructorType.getParameterTypes() :
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   694
                tree.constructor.erasure(types).getParameterTypes();
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   695
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   696
        tree.clazz = translate(tree.clazz, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   697
        if (tree.varargsElement != null)
06bc494ca11e Initial load
duke
parents:
diff changeset
   698
            tree.varargsElement = types.erasure(tree.varargsElement);
06bc494ca11e Initial load
duke
parents:
diff changeset
   699
        tree.args = translateArgs(
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   700
            tree.args, argtypes, tree.varargsElement);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   701
        tree.def = translate(tree.def, null);
27124
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   702
        if (erasedConstructorType != null)
5b69a74c224e 8058199: Code generation problem with javac skipping a checkcast instruction
mcimadamore
parents: 25874
diff changeset
   703
            tree.constructorType = erasedConstructorType;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   704
        tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   705
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   706
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   707
06bc494ca11e Initial load
duke
parents:
diff changeset
   708
    public void visitNewArray(JCNewArray tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   709
        tree.elemtype = translate(tree.elemtype, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   710
        translate(tree.dims, syms.intType);
5008
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   711
        if (tree.type != null) {
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   712
            tree.elems = translate(tree.elems, erasure(types.elemtype(tree.type)));
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   713
            tree.type = erasure(tree.type);
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   714
        } else {
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   715
            tree.elems = translate(tree.elems, null);
3a90203fa91e 6930032: fix findbugs errors in com.sun.tools.javac.comp
jjg
parents: 4865
diff changeset
   716
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   717
06bc494ca11e Initial load
duke
parents:
diff changeset
   718
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   719
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   720
06bc494ca11e Initial load
duke
parents:
diff changeset
   721
    public void visitParens(JCParens tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   722
        tree.expr = translate(tree.expr, pt);
06bc494ca11e Initial load
duke
parents:
diff changeset
   723
        tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   724
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   725
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   726
06bc494ca11e Initial load
duke
parents:
diff changeset
   727
    public void visitAssign(JCAssign tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   728
        tree.lhs = translate(tree.lhs, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   729
        tree.rhs = translate(tree.rhs, erasure(tree.lhs.type));
17810
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   730
        tree.type = erasure(tree.lhs.type);
db9f3e9cd760 7053059: VerifyError with double Assignment using a Generic Member of a Superclass
vromero
parents: 16974
diff changeset
   731
        result = retype(tree, tree.type, pt);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   732
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   733
06bc494ca11e Initial load
duke
parents:
diff changeset
   734
    public void visitAssignop(JCAssignOp tree) {
3558
56be54d60f36 6869075: regression: javac crashes when compiling compound string assignment with generics
mcimadamore
parents: 3542
diff changeset
   735
        tree.lhs = translate(tree.lhs, null);
1468
19d266637353 6751514: Unary post-increment with type variables crash javac during lowering
mcimadamore
parents: 1356
diff changeset
   736
        tree.rhs = translate(tree.rhs, tree.operator.type.getParameterTypes().tail.head);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   737
        tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   738
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   739
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   740
06bc494ca11e Initial load
duke
parents:
diff changeset
   741
    public void visitUnary(JCUnary tree) {
28337
0b46fb7be75a 8043741: VerifyError due to missing checkcast
mcimadamore
parents: 27224
diff changeset
   742
        tree.arg = translate(tree.arg, (tree.getTag() == Tag.NULLCHK)
0b46fb7be75a 8043741: VerifyError due to missing checkcast
mcimadamore
parents: 27224
diff changeset
   743
            ? tree.type
0b46fb7be75a 8043741: VerifyError due to missing checkcast
mcimadamore
parents: 27224
diff changeset
   744
            : tree.operator.type.getParameterTypes().head);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   745
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   746
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   747
06bc494ca11e Initial load
duke
parents:
diff changeset
   748
    public void visitBinary(JCBinary tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   749
        tree.lhs = translate(tree.lhs, tree.operator.type.getParameterTypes().head);
06bc494ca11e Initial load
duke
parents:
diff changeset
   750
        tree.rhs = translate(tree.rhs, tree.operator.type.getParameterTypes().tail.head);
06bc494ca11e Initial load
duke
parents:
diff changeset
   751
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   752
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   753
06bc494ca11e Initial load
duke
parents:
diff changeset
   754
    public void visitTypeCast(JCTypeCast tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   755
        tree.clazz = translate(tree.clazz, null);
16974
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   756
        Type originalTarget = tree.type;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   757
        tree.type = erasure(tree.type);
28339
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   758
        JCExpression newExpression = translate(tree.expr, erasure(tree.expr.type));
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   759
        if (newExpression != tree.expr) {
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   760
            JCTypeCast typeCast = newExpression.hasTag(Tag.TYPECAST)
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   761
                ? (JCTypeCast) newExpression
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   762
                : null;
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   763
            tree.expr = typeCast != null && types.isSameType(typeCast.type, originalTarget, true)
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   764
                ? typeCast.expr
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   765
                : newExpression;
fa0685d84b74 8067914: Redundant type cast nodes in AST (follow up from JDK-8043741)
mcimadamore
parents: 28337
diff changeset
   766
        }
16974
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   767
        if (originalTarget.isCompound()) {
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   768
            Type.IntersectionClassType ict = (Type.IntersectionClassType)originalTarget;
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   769
            for (Type c : ict.getExplicitComponents()) {
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   770
                Type ec = erasure(c);
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   771
                if (!types.isSameType(ec, tree.type)) {
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   772
                    tree.expr = coerce(tree.expr, ec);
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   773
                }
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   774
            }
0c9f2cc31ae7 8011392: Missing checkcast when casting to intersection type
mcimadamore
parents: 16968
diff changeset
   775
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   776
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   777
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   778
06bc494ca11e Initial load
duke
parents:
diff changeset
   779
    public void visitTypeTest(JCInstanceOf tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   780
        tree.expr = translate(tree.expr, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   781
        tree.clazz = translate(tree.clazz, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   782
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   783
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   784
06bc494ca11e Initial load
duke
parents:
diff changeset
   785
    public void visitIndexed(JCArrayAccess tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   786
        tree.indexed = translate(tree.indexed, erasure(tree.indexed.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   787
        tree.index = translate(tree.index, syms.intType);
06bc494ca11e Initial load
duke
parents:
diff changeset
   788
06bc494ca11e Initial load
duke
parents:
diff changeset
   789
        // Insert casts of indexed expressions as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   790
        result = retype(tree, types.elemtype(tree.indexed.type), pt);
06bc494ca11e Initial load
duke
parents:
diff changeset
   791
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   792
06bc494ca11e Initial load
duke
parents:
diff changeset
   793
    // There ought to be nothing to rewrite here;
06bc494ca11e Initial load
duke
parents:
diff changeset
   794
    // we don't generate code.
06bc494ca11e Initial load
duke
parents:
diff changeset
   795
    public void visitAnnotation(JCAnnotation tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   796
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   797
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   798
06bc494ca11e Initial load
duke
parents:
diff changeset
   799
    public void visitIdent(JCIdent tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   800
        Type et = tree.sym.erasure(types);
06bc494ca11e Initial load
duke
parents:
diff changeset
   801
06bc494ca11e Initial load
duke
parents:
diff changeset
   802
        // Map type variables to their bounds.
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
   803
        if (tree.sym.kind == TYP && tree.sym.type.hasTag(TYPEVAR)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   804
            result = make.at(tree.pos).Type(et);
06bc494ca11e Initial load
duke
parents:
diff changeset
   805
        } else
06bc494ca11e Initial load
duke
parents:
diff changeset
   806
        // Map constants expressions to themselves.
06bc494ca11e Initial load
duke
parents:
diff changeset
   807
        if (tree.type.constValue() != null) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   808
            result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   809
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   810
        // Insert casts of variable uses as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   811
        else if (tree.sym.kind == VAR) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   812
            result = retype(tree, et, pt);
06bc494ca11e Initial load
duke
parents:
diff changeset
   813
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   814
        else {
06bc494ca11e Initial load
duke
parents:
diff changeset
   815
            tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   816
            result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   817
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   818
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   819
06bc494ca11e Initial load
duke
parents:
diff changeset
   820
    public void visitSelect(JCFieldAccess tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   821
        Type t = tree.selected.type;
14359
d4099818ab70 7200915: convert TypeTags from a series of small ints to an enum
jjg
parents: 14259
diff changeset
   822
        while (t.hasTag(TYPEVAR))
507
bdd92cc35ac4 6531075: Missing synthetic casts when accessing fields/methods of intersection types including type variables
mcimadamore
parents: 10
diff changeset
   823
            t = t.getUpperBound();
bdd92cc35ac4 6531075: Missing synthetic casts when accessing fields/methods of intersection types including type variables
mcimadamore
parents: 10
diff changeset
   824
        if (t.isCompound()) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   825
            if ((tree.sym.flags() & IPROXY) != 0) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   826
                tree.sym = ((MethodSymbol)tree.sym).
06bc494ca11e Initial load
duke
parents:
diff changeset
   827
                    implemented((TypeSymbol)tree.sym.owner, types);
06bc494ca11e Initial load
duke
parents:
diff changeset
   828
            }
7617
6d4424dd7256 6956758: NPE in com.sun.tools.javac.code.Symbol - isSubClass
mcimadamore
parents: 7080
diff changeset
   829
            tree.selected = coerce(
507
bdd92cc35ac4 6531075: Missing synthetic casts when accessing fields/methods of intersection types including type variables
mcimadamore
parents: 10
diff changeset
   830
                translate(tree.selected, erasure(tree.selected.type)),
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   831
                erasure(tree.sym.owner.type));
06bc494ca11e Initial load
duke
parents:
diff changeset
   832
        } else
06bc494ca11e Initial load
duke
parents:
diff changeset
   833
            tree.selected = translate(tree.selected, erasure(t));
06bc494ca11e Initial load
duke
parents:
diff changeset
   834
06bc494ca11e Initial load
duke
parents:
diff changeset
   835
        // Map constants expressions to themselves.
06bc494ca11e Initial load
duke
parents:
diff changeset
   836
        if (tree.type.constValue() != null) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   837
            result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   838
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   839
        // Insert casts of variable uses as needed.
06bc494ca11e Initial load
duke
parents:
diff changeset
   840
        else if (tree.sym.kind == VAR) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   841
            result = retype(tree, tree.sym.erasure(types), pt);
06bc494ca11e Initial load
duke
parents:
diff changeset
   842
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   843
        else {
06bc494ca11e Initial load
duke
parents:
diff changeset
   844
            tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   845
            result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   846
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   847
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   848
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   849
    public void visitReference(JCMemberReference tree) {
19939
ab90fbc7b74e 8023558: Javac creates invalid bootstrap methods for complex lambda/methodref case
vromero
parents: 19133
diff changeset
   850
        tree.expr = translate(tree.expr, erasure(tree.expr.type));
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   851
        tree.type = erasure(tree.type);
23127
5aba53c2b9b2 8034048: javac crash with method references plus lambda plus var args
jjg
parents: 22168
diff changeset
   852
        if (tree.varargsElement != null)
5aba53c2b9b2 8034048: javac crash with method references plus lambda plus var args
jjg
parents: 22168
diff changeset
   853
            tree.varargsElement = erasure(tree.varargsElement);
14365
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   854
        result = tree;
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   855
    }
20f388573215 8000694: Add generation of lambda implementation code: invokedynamic call, lambda method, adaptor methods
rfield
parents: 14359
diff changeset
   856
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   857
    public void visitTypeArray(JCArrayTypeTree tree) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   858
        tree.elemtype = translate(tree.elemtype, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   859
        tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   860
        result = tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   861
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   862
06bc494ca11e Initial load
duke
parents:
diff changeset
   863
    /** Visitor method for parameterized types.
06bc494ca11e Initial load
duke
parents:
diff changeset
   864
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   865
    public void visitTypeApply(JCTypeApply tree) {
3149
0cd06d598d6f 6843077: JSR 308: Annotations on types
jjg
parents: 1468
diff changeset
   866
        JCTree clazz = translate(tree.clazz, null);
0cd06d598d6f 6843077: JSR 308: Annotations on types
jjg
parents: 1468
diff changeset
   867
        result = clazz;
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   868
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   869
14725
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   870
    public void visitTypeIntersection(JCTypeIntersection tree) {
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   871
        tree.bounds = translate(tree.bounds, null);
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   872
        tree.type = erasure(tree.type);
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   873
        result = tree;
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   874
    }
65836e833f59 8002099: Add support for intersection types in cast expression
mcimadamore
parents: 14547
diff changeset
   875
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   876
/**************************************************************************
06bc494ca11e Initial load
duke
parents:
diff changeset
   877
 * utility methods
06bc494ca11e Initial load
duke
parents:
diff changeset
   878
 *************************************************************************/
06bc494ca11e Initial load
duke
parents:
diff changeset
   879
06bc494ca11e Initial load
duke
parents:
diff changeset
   880
    private Type erasure(Type t) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   881
        return types.erasure(t);
06bc494ca11e Initial load
duke
parents:
diff changeset
   882
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   883
06bc494ca11e Initial load
duke
parents:
diff changeset
   884
/**************************************************************************
06bc494ca11e Initial load
duke
parents:
diff changeset
   885
 * main method
06bc494ca11e Initial load
duke
parents:
diff changeset
   886
 *************************************************************************/
06bc494ca11e Initial load
duke
parents:
diff changeset
   887
06bc494ca11e Initial load
duke
parents:
diff changeset
   888
    private Env<AttrContext> env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   889
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   890
    private static final String statePreviousToFlowAssertMsg =
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   891
            "The current compile state [%s] of class %s is previous to FLOW";
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   892
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   893
    void translateClass(ClassSymbol c) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   894
        Type st = types.supertype(c.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   895
        // process superclass before derived
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   896
        if (st.hasTag(CLASS)) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   897
            translateClass((ClassSymbol)st.tsym);
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   898
        }
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   899
25007
eb097d3a68f5 8038975: Access control in enhanced for
pgovereau
parents: 24793
diff changeset
   900
        Env<AttrContext> myEnv = enter.getEnv(c);
eb097d3a68f5 8038975: Access control in enhanced for
pgovereau
parents: 24793
diff changeset
   901
        if (myEnv == null || (c.flags_field & TYPE_TRANSLATED) != 0) {
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   902
            return;
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   903
        }
25007
eb097d3a68f5 8038975: Access control in enhanced for
pgovereau
parents: 24793
diff changeset
   904
        c.flags_field |= TYPE_TRANSLATED;
16968
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   905
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   906
        /*  The two assertions below are set for early detection of any attempt
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   907
         *  to translate a class that:
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   908
         *
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   909
         *  1) has no compile state being it the most outer class.
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   910
         *     We accept this condition for inner classes.
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   911
         *
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   912
         *  2) has a compile state which is previous to Flow state.
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   913
         */
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   914
        boolean envHasCompState = compileStates.get(myEnv) != null;
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   915
        if (!envHasCompState && c.outermostClass() == c) {
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   916
            Assert.error("No info for outermost class: " + myEnv.enclClass.sym);
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   917
        }
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   918
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   919
        if (envHasCompState &&
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   920
                CompileState.FLOW.isAfter(compileStates.get(myEnv))) {
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   921
            Assert.error(String.format(statePreviousToFlowAssertMsg,
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   922
                    compileStates.get(myEnv), myEnv.enclClass.sym));
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   923
        }
19f0da2d3143 8010659: Javac Crashes while building OpenJFX
vromero
parents: 15385
diff changeset
   924
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   925
        Env<AttrContext> oldEnv = env;
06bc494ca11e Initial load
duke
parents:
diff changeset
   926
        try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   927
            env = myEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   928
            // class has not been translated yet
06bc494ca11e Initial load
duke
parents:
diff changeset
   929
06bc494ca11e Initial load
duke
parents:
diff changeset
   930
            TreeMaker savedMake = make;
06bc494ca11e Initial load
duke
parents:
diff changeset
   931
            Type savedPt = pt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   932
            make = make.forToplevel(env.toplevel);
06bc494ca11e Initial load
duke
parents:
diff changeset
   933
            pt = null;
06bc494ca11e Initial load
duke
parents:
diff changeset
   934
            try {
06bc494ca11e Initial load
duke
parents:
diff changeset
   935
                JCClassDecl tree = (JCClassDecl) env.tree;
06bc494ca11e Initial load
duke
parents:
diff changeset
   936
                tree.typarams = List.nil();
06bc494ca11e Initial load
duke
parents:
diff changeset
   937
                super.visitClassDef(tree);
06bc494ca11e Initial load
duke
parents:
diff changeset
   938
                make.at(tree.pos);
25445
603f0c93d5c9 8011044: Remove support for 1.5 and earlier source and target options
ntoda
parents: 25443
diff changeset
   939
                ListBuffer<JCTree> bridges = new ListBuffer<>();
603f0c93d5c9 8011044: Remove support for 1.5 and earlier source and target options
ntoda
parents: 25443
diff changeset
   940
                if (allowInterfaceBridges || (tree.sym.flags() & INTERFACE) == 0) {
603f0c93d5c9 8011044: Remove support for 1.5 and earlier source and target options
ntoda
parents: 25443
diff changeset
   941
                    addBridges(tree.pos(), c, bridges);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   942
                }
25445
603f0c93d5c9 8011044: Remove support for 1.5 and earlier source and target options
ntoda
parents: 25443
diff changeset
   943
                tree.defs = bridges.toList().prependList(tree.defs);
10
06bc494ca11e Initial load
duke
parents:
diff changeset
   944
                tree.type = erasure(tree.type);
06bc494ca11e Initial load
duke
parents:
diff changeset
   945
            } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   946
                make = savedMake;
06bc494ca11e Initial load
duke
parents:
diff changeset
   947
                pt = savedPt;
06bc494ca11e Initial load
duke
parents:
diff changeset
   948
            }
06bc494ca11e Initial load
duke
parents:
diff changeset
   949
        } finally {
06bc494ca11e Initial load
duke
parents:
diff changeset
   950
            env = oldEnv;
06bc494ca11e Initial load
duke
parents:
diff changeset
   951
        }
06bc494ca11e Initial load
duke
parents:
diff changeset
   952
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   953
06bc494ca11e Initial load
duke
parents:
diff changeset
   954
    /** Translate a toplevel class definition.
06bc494ca11e Initial load
duke
parents:
diff changeset
   955
     *  @param cdef    The definition to be translated.
06bc494ca11e Initial load
duke
parents:
diff changeset
   956
     */
06bc494ca11e Initial load
duke
parents:
diff changeset
   957
    public JCTree translateTopLevelClass(JCTree cdef, TreeMaker make) {
06bc494ca11e Initial load
duke
parents:
diff changeset
   958
        // note that this method does NOT support recursion.
06bc494ca11e Initial load
duke
parents:
diff changeset
   959
        this.make = make;
06bc494ca11e Initial load
duke
parents:
diff changeset
   960
        pt = null;
06bc494ca11e Initial load
duke
parents:
diff changeset
   961
        return translate(cdef, null);
06bc494ca11e Initial load
duke
parents:
diff changeset
   962
    }
06bc494ca11e Initial load
duke
parents:
diff changeset
   963
}