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