author | mcimadamore |
Fri, 16 Dec 2016 15:27:34 +0000 | |
changeset 42827 | 36468b5fa7f4 |
parent 38516 | b643c42e9d25 |
child 44186 | fe848a208b20 |
permissions | -rw-r--r-- |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
1 |
/* |
28891
dc7d89f2e25c
8069545: javac shouldn't check nested stuck lambdas during overload resolution
sadayapalam
parents:
28458
diff
changeset
|
2 |
* Copyright (c) 2012, 2015, Oracle and/or its affiliates. All rights reserved. |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
4 |
* |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. Oracle designates this |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
8 |
* particular file as subject to the "Classpath" exception as provided |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
10 |
* |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
15 |
* accompanied this code). |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
16 |
* |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
17 |
* You should have received a copy of the GNU General Public License version |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
20 |
* |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
22 |
* or visit www.oracle.com if you need additional information or have any |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
23 |
* questions. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
24 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
25 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
26 |
package com.sun.tools.javac.comp; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
27 |
|
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
28 |
import com.sun.source.tree.LambdaExpressionTree.BodyKind; |
29776
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
29 |
import com.sun.source.tree.NewClassTree; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
30 |
import com.sun.tools.javac.code.*; |
29554
6d7957bd6866
8074100: Turn Type.Mapping into a true visitor
mcimadamore
parents:
29292
diff
changeset
|
31 |
import com.sun.tools.javac.code.Type.TypeMapping; |
32911
6ee3c12d2d18
8137269: Add better support for local caching in ArgumentAttr
mcimadamore
parents:
32709
diff
changeset
|
32 |
import com.sun.tools.javac.comp.ArgumentAttr.LocalCacheContext; |
28456
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
33 |
import com.sun.tools.javac.comp.Resolve.ResolveError; |
28458
f20c5dd05e34
8069164: Fix langtools make build so that diagnostic framework can be used
mcimadamore
parents:
28457
diff
changeset
|
34 |
import com.sun.tools.javac.resources.CompilerProperties.Fragments; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
35 |
import com.sun.tools.javac.tree.*; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
36 |
import com.sun.tools.javac.util.*; |
26266
2d24bda701dc
8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents:
25874
diff
changeset
|
37 |
import com.sun.tools.javac.util.DefinedBy.Api; |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
38 |
import com.sun.tools.javac.util.GraphUtils.DependencyKind; |
16805
b419336382fd
8010922: Cleanup: add support for ad-hoc method check logic
mcimadamore
parents:
16566
diff
changeset
|
39 |
import com.sun.tools.javac.util.JCDiagnostic.DiagnosticPosition; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
40 |
import com.sun.tools.javac.code.Symbol.*; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
41 |
import com.sun.tools.javac.comp.Attr.ResultInfo; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
42 |
import com.sun.tools.javac.comp.Resolve.MethodResolutionPhase; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
43 |
import com.sun.tools.javac.tree.JCTree.*; |
28456
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
44 |
import com.sun.tools.javac.util.JCDiagnostic.DiagnosticType; |
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
45 |
import com.sun.tools.javac.util.Log.DeferredDiagnosticHandler; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
46 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
47 |
import java.util.ArrayList; |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
48 |
import java.util.Collection; |
19914 | 49 |
import java.util.Collections; |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
50 |
import java.util.EnumSet; |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
51 |
import java.util.HashSet; |
14547 | 52 |
import java.util.LinkedHashSet; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
53 |
import java.util.Map; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
54 |
import java.util.Set; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
55 |
import java.util.WeakHashMap; |
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
56 |
import java.util.function.Function; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
57 |
|
14547 | 58 |
import static com.sun.tools.javac.code.TypeTag.*; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
59 |
import static com.sun.tools.javac.tree.JCTree.Tag.*; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
60 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
61 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
62 |
* This is an helper class that is used to perform deferred type-analysis. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
63 |
* Each time a poly expression occurs in argument position, javac attributes it |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
64 |
* with a temporary 'deferred type' that is checked (possibly multiple times) |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
65 |
* against an expected formal type. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
66 |
* |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
67 |
* <p><b>This is NOT part of any supported API. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
68 |
* If you write code that depends on this, you do so at your own risk. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
69 |
* This code and its internal interfaces are subject to change or |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
70 |
* deletion without notice.</b> |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
71 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
72 |
public class DeferredAttr extends JCTree.Visitor { |
22163 | 73 |
protected static final Context.Key<DeferredAttr> deferredAttrKey = new Context.Key<>(); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
74 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
75 |
final Attr attr; |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
76 |
final ArgumentAttr argumentAttr; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
77 |
final Check chk; |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
78 |
final JCDiagnostic.Factory diags; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
79 |
final Enter enter; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
80 |
final Infer infer; |
16294
0c291a3cd60d
8007462: Fix provisional applicability for method references
mcimadamore
parents:
15717
diff
changeset
|
81 |
final Resolve rs; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
82 |
final Log log; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
83 |
final Symtab syms; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
84 |
final TreeMaker make; |
29776
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
85 |
final TreeCopier<Void> treeCopier; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
86 |
final TypeMapping<Void> deferredCopier; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
87 |
final Types types; |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
88 |
final Flow flow; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
89 |
final Names names; |
25007 | 90 |
final TypeEnvs typeEnvs; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
91 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
92 |
public static DeferredAttr instance(Context context) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
93 |
DeferredAttr instance = context.get(deferredAttrKey); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
94 |
if (instance == null) |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
95 |
instance = new DeferredAttr(context); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
96 |
return instance; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
97 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
98 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
99 |
protected DeferredAttr(Context context) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
100 |
context.put(deferredAttrKey, this); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
101 |
attr = Attr.instance(context); |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
102 |
argumentAttr = ArgumentAttr.instance(context); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
103 |
chk = Check.instance(context); |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
104 |
diags = JCDiagnostic.Factory.instance(context); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
105 |
enter = Enter.instance(context); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
106 |
infer = Infer.instance(context); |
16294
0c291a3cd60d
8007462: Fix provisional applicability for method references
mcimadamore
parents:
15717
diff
changeset
|
107 |
rs = Resolve.instance(context); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
108 |
log = Log.instance(context); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
109 |
syms = Symtab.instance(context); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
110 |
make = TreeMaker.instance(context); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
111 |
types = Types.instance(context); |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
112 |
flow = Flow.instance(context); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
113 |
names = Names.instance(context); |
18902 | 114 |
stuckTree = make.Ident(names.empty).setType(Type.stuckType); |
25007 | 115 |
typeEnvs = TypeEnvs.instance(context); |
18910
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
116 |
emptyDeferredAttrContext = |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
117 |
new DeferredAttrContext(AttrMode.CHECK, null, MethodResolutionPhase.BOX, infer.emptyContext, null, null) { |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
118 |
@Override |
19914 | 119 |
void addDeferredAttrNode(DeferredType dt, ResultInfo ri, DeferredStuckPolicy deferredStuckPolicy) { |
18910
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
120 |
Assert.error("Empty deferred context!"); |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
121 |
} |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
122 |
@Override |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
123 |
void complete() { |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
124 |
Assert.error("Empty deferred context!"); |
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
125 |
} |
24226
08b586e22328
8030741: Inference: implement eager resolution of return types, consistent with JDK-8028800
vromero
parents:
24066
diff
changeset
|
126 |
|
08b586e22328
8030741: Inference: implement eager resolution of return types, consistent with JDK-8028800
vromero
parents:
24066
diff
changeset
|
127 |
@Override |
08b586e22328
8030741: Inference: implement eager resolution of return types, consistent with JDK-8028800
vromero
parents:
24066
diff
changeset
|
128 |
public String toString() { |
08b586e22328
8030741: Inference: implement eager resolution of return types, consistent with JDK-8028800
vromero
parents:
24066
diff
changeset
|
129 |
return "Empty deferred context!"; |
08b586e22328
8030741: Inference: implement eager resolution of return types, consistent with JDK-8028800
vromero
parents:
24066
diff
changeset
|
130 |
} |
18910
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
131 |
}; |
29776
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
132 |
|
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
133 |
// For speculative attribution, skip the class definition in <>. |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
134 |
treeCopier = |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
135 |
new TreeCopier<Void>(make) { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
136 |
@Override @DefinedBy(Api.COMPILER_TREE) |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
137 |
public JCTree visitNewClass(NewClassTree node, Void p) { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
138 |
JCNewClass t = (JCNewClass) node; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
139 |
if (TreeInfo.isDiamond(t)) { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
140 |
JCExpression encl = copy(t.encl, p); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
141 |
List<JCExpression> typeargs = copy(t.typeargs, p); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
142 |
JCExpression clazz = copy(t.clazz, p); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
143 |
List<JCExpression> args = copy(t.args, p); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
144 |
JCClassDecl def = null; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
145 |
return make.at(t.pos).NewClass(encl, typeargs, clazz, args, def); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
146 |
} else { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
147 |
return super.visitNewClass(node, p); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
148 |
} |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
149 |
} |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
150 |
}; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
151 |
deferredCopier = new TypeMapping<Void> () { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
152 |
@Override |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
153 |
public Type visitType(Type t, Void v) { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
154 |
if (t.hasTag(DEFERRED)) { |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
155 |
DeferredType dt = (DeferredType) t; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
156 |
return new DeferredType(treeCopier.copy(dt.tree), dt.env); |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
157 |
} |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
158 |
return t; |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
159 |
} |
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
160 |
}; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
161 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
162 |
|
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
163 |
/** shared tree for stuck expressions */ |
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
164 |
final JCTree stuckTree; |
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
165 |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
166 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
167 |
* This type represents a deferred type. A deferred type starts off with |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
168 |
* no information on the underlying expression type. Such info needs to be |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
169 |
* discovered through type-checking the deferred type against a target-type. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
170 |
* Every deferred type keeps a pointer to the AST node from which it originated. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
171 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
172 |
public class DeferredType extends Type { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
173 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
174 |
public JCExpression tree; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
175 |
Env<AttrContext> env; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
176 |
AttrMode mode; |
38512
c71e1cdd6674
8154180: Regression: stuck expressions do not behave correctly
mcimadamore
parents:
36526
diff
changeset
|
177 |
boolean pertinentToApplicability = true; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
178 |
SpeculativeCache speculativeCache; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
179 |
|
26532
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
180 |
DeferredType(JCExpression tree, Env<AttrContext> env) { |
29842
826ac2519523
8031744: Annotations on many Language Model elements are not returned
jjg
parents:
29776
diff
changeset
|
181 |
super(null, TypeMetadata.EMPTY); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
182 |
this.tree = tree; |
18912
e25cd61d8e59
8020147: Spurious errors when compiling nested stuck lambdas
mcimadamore
parents:
18910
diff
changeset
|
183 |
this.env = attr.copyEnv(env); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
184 |
this.speculativeCache = new SpeculativeCache(); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
185 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
186 |
|
18646
e628560a86d1
8017104: javac should have a class for primitive types that inherits from Type
vromero
parents:
18643
diff
changeset
|
187 |
@Override |
29842
826ac2519523
8031744: Annotations on many Language Model elements are not returned
jjg
parents:
29776
diff
changeset
|
188 |
public DeferredType cloneWithMetadata(TypeMetadata md) { |
26781
a786b07c7b91
8048614: Add TypeMetadata to contain type annotations and other type information
emc
parents:
26532
diff
changeset
|
189 |
throw new AssertionError("Cannot add metadata to a deferred type"); |
24396 | 190 |
} |
191 |
||
192 |
@Override |
|
18646
e628560a86d1
8017104: javac should have a class for primitive types that inherits from Type
vromero
parents:
18643
diff
changeset
|
193 |
public TypeTag getTag() { |
e628560a86d1
8017104: javac should have a class for primitive types that inherits from Type
vromero
parents:
18643
diff
changeset
|
194 |
return DEFERRED; |
e628560a86d1
8017104: javac should have a class for primitive types that inherits from Type
vromero
parents:
18643
diff
changeset
|
195 |
} |
e628560a86d1
8017104: javac should have a class for primitive types that inherits from Type
vromero
parents:
18643
diff
changeset
|
196 |
|
26266
2d24bda701dc
8056061: Mark implementations of public interfaces with an annotation
jlahoda
parents:
25874
diff
changeset
|
197 |
@Override @DefinedBy(Api.LANGUAGE_MODEL) |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
198 |
public String toString() { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
199 |
return "DeferredType"; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
200 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
201 |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
202 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
203 |
* A speculative cache is used to keep track of all overload resolution rounds |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
204 |
* that triggered speculative attribution on a given deferred type. Each entry |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
205 |
* stores a pointer to the speculative tree and the resolution phase in which the entry |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
206 |
* has been added. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
207 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
208 |
class SpeculativeCache { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
209 |
|
22163 | 210 |
private Map<Symbol, List<Entry>> cache = new WeakHashMap<>(); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
211 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
212 |
class Entry { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
213 |
JCTree speculativeTree; |
19914 | 214 |
ResultInfo resultInfo; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
215 |
|
19914 | 216 |
public Entry(JCTree speculativeTree, ResultInfo resultInfo) { |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
217 |
this.speculativeTree = speculativeTree; |
19914 | 218 |
this.resultInfo = resultInfo; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
219 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
220 |
|
19914 | 221 |
boolean matches(MethodResolutionPhase phase) { |
222 |
return resultInfo.checkContext.deferredAttrContext().phase == phase; |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
223 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
224 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
225 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
226 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
227 |
* Retrieve a speculative cache entry corresponding to given symbol |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
228 |
* and resolution phase |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
229 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
230 |
Entry get(Symbol msym, MethodResolutionPhase phase) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
231 |
List<Entry> entries = cache.get(msym); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
232 |
if (entries == null) return null; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
233 |
for (Entry e : entries) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
234 |
if (e.matches(phase)) return e; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
235 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
236 |
return null; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
237 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
238 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
239 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
240 |
* Stores a speculative cache entry corresponding to given symbol |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
241 |
* and resolution phase |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
242 |
*/ |
19914 | 243 |
void put(JCTree speculativeTree, ResultInfo resultInfo) { |
244 |
Symbol msym = resultInfo.checkContext.deferredAttrContext().msym; |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
245 |
List<Entry> entries = cache.get(msym); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
246 |
if (entries == null) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
247 |
entries = List.nil(); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
248 |
} |
19914 | 249 |
cache.put(msym, entries.prepend(new Entry(speculativeTree, resultInfo))); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
250 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
251 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
252 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
253 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
254 |
* Get the type that has been computed during a speculative attribution round |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
255 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
256 |
Type speculativeType(Symbol msym, MethodResolutionPhase phase) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
257 |
SpeculativeCache.Entry e = speculativeCache.get(msym, phase); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
258 |
return e != null ? e.speculativeTree.type : Type.noType; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
259 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
260 |
|
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
261 |
JCTree speculativeTree(DeferredAttrContext deferredAttrContext) { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
262 |
DeferredType.SpeculativeCache.Entry e = speculativeCache.get(deferredAttrContext.msym, deferredAttrContext.phase); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
263 |
return e != null ? e.speculativeTree : stuckTree; |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
264 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
265 |
|
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
266 |
DeferredTypeCompleter completer() { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
267 |
return basicCompleter; |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
268 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
269 |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
270 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
271 |
* Check a deferred type against a potential target-type. Depending on |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
272 |
* the current attribution mode, a normal vs. speculative attribution |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
273 |
* round is performed on the underlying AST node. There can be only one |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
274 |
* speculative round for a given target method symbol; moreover, a normal |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
275 |
* attribution round must follow one or more speculative rounds. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
276 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
277 |
Type check(ResultInfo resultInfo) { |
19914 | 278 |
DeferredStuckPolicy deferredStuckPolicy; |
279 |
if (resultInfo.pt.hasTag(NONE) || resultInfo.pt.isErroneous()) { |
|
280 |
deferredStuckPolicy = dummyStuckPolicy; |
|
28891
dc7d89f2e25c
8069545: javac shouldn't check nested stuck lambdas during overload resolution
sadayapalam
parents:
28458
diff
changeset
|
281 |
} else if (resultInfo.checkContext.deferredAttrContext().mode == AttrMode.SPECULATIVE || |
dc7d89f2e25c
8069545: javac shouldn't check nested stuck lambdas during overload resolution
sadayapalam
parents:
28458
diff
changeset
|
282 |
resultInfo.checkContext.deferredAttrContext().insideOverloadPhase()) { |
19914 | 283 |
deferredStuckPolicy = new OverloadStuckPolicy(resultInfo, this); |
284 |
} else { |
|
285 |
deferredStuckPolicy = new CheckStuckPolicy(resultInfo, this); |
|
286 |
} |
|
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
287 |
return check(resultInfo, deferredStuckPolicy, completer()); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
288 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
289 |
|
19914 | 290 |
private Type check(ResultInfo resultInfo, DeferredStuckPolicy deferredStuckPolicy, |
291 |
DeferredTypeCompleter deferredTypeCompleter) { |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
292 |
DeferredAttrContext deferredAttrContext = |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
293 |
resultInfo.checkContext.deferredAttrContext(); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
294 |
Assert.check(deferredAttrContext != emptyDeferredAttrContext); |
19914 | 295 |
if (deferredStuckPolicy.isStuck()) { |
38512
c71e1cdd6674
8154180: Regression: stuck expressions do not behave correctly
mcimadamore
parents:
36526
diff
changeset
|
296 |
pertinentToApplicability = false; |
19914 | 297 |
deferredAttrContext.addDeferredAttrNode(this, resultInfo, deferredStuckPolicy); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
298 |
return Type.noType; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
299 |
} else { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
300 |
try { |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
301 |
return deferredTypeCompleter.complete(this, resultInfo, deferredAttrContext); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
302 |
} finally { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
303 |
mode = deferredAttrContext.mode; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
304 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
305 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
306 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
307 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
308 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
309 |
/** |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
310 |
* A completer for deferred types. Defines an entry point for type-checking |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
311 |
* a deferred type. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
312 |
*/ |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
313 |
interface DeferredTypeCompleter { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
314 |
/** |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
315 |
* Entry point for type-checking a deferred type. Depending on the |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
316 |
* circumstances, type-checking could amount to full attribution |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
317 |
* or partial structural check (aka potential applicability). |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
318 |
*/ |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
319 |
Type complete(DeferredType dt, ResultInfo resultInfo, DeferredAttrContext deferredAttrContext); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
320 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
321 |
|
19914 | 322 |
|
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
323 |
/** |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
324 |
* A basic completer for deferred types. This completer type-checks a deferred type |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
325 |
* using attribution; depending on the attribution mode, this could be either standard |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
326 |
* or speculative attribution. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
327 |
*/ |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
328 |
DeferredTypeCompleter basicCompleter = new DeferredTypeCompleter() { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
329 |
public Type complete(DeferredType dt, ResultInfo resultInfo, DeferredAttrContext deferredAttrContext) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
330 |
switch (deferredAttrContext.mode) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
331 |
case SPECULATIVE: |
16566
604ce45d5f23
8009820: AssertionError when compiling java code with two identical static imports
mcimadamore
parents:
16326
diff
changeset
|
332 |
//Note: if a symbol is imported twice we might do two identical |
604ce45d5f23
8009820: AssertionError when compiling java code with two identical static imports
mcimadamore
parents:
16326
diff
changeset
|
333 |
//speculative rounds... |
604ce45d5f23
8009820: AssertionError when compiling java code with two identical static imports
mcimadamore
parents:
16326
diff
changeset
|
334 |
Assert.check(dt.mode == null || dt.mode == AttrMode.SPECULATIVE); |
26532
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
335 |
JCTree speculativeTree = attribSpeculative(dt.tree, dt.env, resultInfo); |
19914 | 336 |
dt.speculativeCache.put(speculativeTree, resultInfo); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
337 |
return speculativeTree.type; |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
338 |
case CHECK: |
15717 | 339 |
Assert.check(dt.mode != null); |
26532
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
340 |
return attr.attribTree(dt.tree, dt.env, resultInfo); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
341 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
342 |
Assert.error(); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
343 |
return null; |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
344 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
345 |
}; |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
346 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
347 |
/** |
19914 | 348 |
* Policy for detecting stuck expressions. Different criteria might cause |
349 |
* an expression to be judged as stuck, depending on whether the check |
|
350 |
* is performed during overload resolution or after most specific. |
|
351 |
*/ |
|
352 |
interface DeferredStuckPolicy { |
|
353 |
/** |
|
354 |
* Has the policy detected that a given expression should be considered stuck? |
|
355 |
*/ |
|
356 |
boolean isStuck(); |
|
357 |
/** |
|
358 |
* Get the set of inference variables a given expression depends upon. |
|
359 |
*/ |
|
360 |
Set<Type> stuckVars(); |
|
361 |
/** |
|
362 |
* Get the set of inference variables which might get new constraints |
|
363 |
* if a given expression is being type-checked. |
|
364 |
*/ |
|
365 |
Set<Type> depVars(); |
|
366 |
} |
|
367 |
||
368 |
/** |
|
369 |
* Basic stuck policy; an expression is never considered to be stuck. |
|
370 |
*/ |
|
371 |
DeferredStuckPolicy dummyStuckPolicy = new DeferredStuckPolicy() { |
|
372 |
@Override |
|
373 |
public boolean isStuck() { |
|
374 |
return false; |
|
375 |
} |
|
376 |
@Override |
|
377 |
public Set<Type> stuckVars() { |
|
378 |
return Collections.emptySet(); |
|
379 |
} |
|
380 |
@Override |
|
381 |
public Set<Type> depVars() { |
|
382 |
return Collections.emptySet(); |
|
383 |
} |
|
384 |
}; |
|
385 |
||
386 |
/** |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
387 |
* The 'mode' in which the deferred type is to be type-checked |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
388 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
389 |
public enum AttrMode { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
390 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
391 |
* A speculative type-checking round is used during overload resolution |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
392 |
* mainly to generate constraints on inference variables. Side-effects |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
393 |
* arising from type-checking the expression associated with the deferred |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
394 |
* type are reversed after the speculative round finishes. This means the |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
395 |
* expression tree will be left in a blank state. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
396 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
397 |
SPECULATIVE, |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
398 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
399 |
* This is the plain type-checking mode. Produces side-effects on the underlying AST node |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
400 |
*/ |
22163 | 401 |
CHECK |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
402 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
403 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
404 |
/** |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
405 |
* Performs speculative attribution of a lambda body and returns the speculative lambda tree, |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
406 |
* in the absence of a target-type. Since {@link Attr#visitLambda(JCLambda)} cannot type-check |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
407 |
* lambda bodies w/o a suitable target-type, this routine 'unrolls' the lambda by turning it |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
408 |
* into a regular block, speculatively type-checks the block and then puts back the pieces. |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
409 |
*/ |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
410 |
JCLambda attribSpeculativeLambda(JCLambda that, Env<AttrContext> env, ResultInfo resultInfo) { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
411 |
ListBuffer<JCStatement> stats = new ListBuffer<>(); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
412 |
stats.addAll(that.params); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
413 |
if (that.getBodyKind() == JCLambda.BodyKind.EXPRESSION) { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
414 |
stats.add(make.Return((JCExpression)that.body)); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
415 |
} else { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
416 |
stats.add((JCBlock)that.body); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
417 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
418 |
JCBlock lambdaBlock = make.Block(0, stats.toList()); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
419 |
Env<AttrContext> localEnv = attr.lambdaEnv(that, env); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
420 |
try { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
421 |
localEnv.info.returnResult = resultInfo; |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
422 |
JCBlock speculativeTree = (JCBlock)attribSpeculative(lambdaBlock, localEnv, resultInfo); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
423 |
List<JCVariableDecl> args = speculativeTree.getStatements().stream() |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
424 |
.filter(s -> s.hasTag(Tag.VARDEF)) |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
425 |
.map(t -> (JCVariableDecl)t) |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
426 |
.collect(List.collector()); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
427 |
JCTree lambdaBody = speculativeTree.getStatements().last(); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
428 |
if (lambdaBody.hasTag(Tag.RETURN)) { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
429 |
lambdaBody = ((JCReturn)lambdaBody).expr; |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
430 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
431 |
JCLambda speculativeLambda = make.Lambda(args, lambdaBody); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
432 |
attr.preFlow(speculativeLambda); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
433 |
flow.analyzeLambda(env, speculativeLambda, make, false); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
434 |
return speculativeLambda; |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
435 |
} finally { |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
436 |
localEnv.info.scope.leave(); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
437 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
438 |
} |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
439 |
|
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
440 |
/** |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
441 |
* Routine that performs speculative type-checking; the input AST node is |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
442 |
* cloned (to avoid side-effects cause by Attr) and compiler state is |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
443 |
* restored after type-checking. All diagnostics (but critical ones) are |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
444 |
* disabled during speculative type-checking. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
445 |
*/ |
26532
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
446 |
JCTree attribSpeculative(JCTree tree, Env<AttrContext> env, ResultInfo resultInfo) { |
29776
984a79b71cfe
8062373: Project Coin: diamond and anonymous classes
sadayapalam
parents:
29774
diff
changeset
|
447 |
return attribSpeculative(tree, env, resultInfo, treeCopier, |
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
448 |
(newTree)->new DeferredAttrDiagHandler(log, newTree)); |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
449 |
} |
16309
db71f8ecc136
8008276: assertion error in com.sun.tools.javac.comp.TransTypes.visitApply
mcimadamore
parents:
16294
diff
changeset
|
450 |
|
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
451 |
<Z> JCTree attribSpeculative(JCTree tree, Env<AttrContext> env, ResultInfo resultInfo, TreeCopier<Z> deferredCopier, |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
452 |
Function<JCTree, DeferredDiagnosticHandler> diagHandlerCreator) { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
453 |
final JCTree newTree = deferredCopier.copy(tree); |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
454 |
Env<AttrContext> speculativeEnv = env.dup(newTree, env.info.dup(env.info.scope.dupUnshared(env.info.scope.owner))); |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
455 |
speculativeEnv.info.isSpeculative = true; |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
456 |
Log.DeferredDiagnosticHandler deferredDiagnosticHandler = diagHandlerCreator.apply(newTree); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
457 |
try { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
458 |
attr.attribTree(newTree, speculativeEnv, resultInfo); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
459 |
return newTree; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
460 |
} finally { |
36526 | 461 |
new UnenterScanner(env.toplevel.modle).scan(newTree); |
14538
384681be798f
8003299: Cleanup javac Log support for deferred diagnostics
jjg
parents:
14359
diff
changeset
|
462 |
log.popDiagnosticHandler(deferredDiagnosticHandler); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
463 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
464 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
465 |
//where |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
466 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
467 |
class UnenterScanner extends TreeScanner { |
36526 | 468 |
private final ModuleSymbol msym; |
469 |
||
470 |
public UnenterScanner(ModuleSymbol msym) { |
|
471 |
this.msym = msym; |
|
472 |
} |
|
473 |
||
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
474 |
@Override |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
475 |
public void visitClassDef(JCClassDecl tree) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
476 |
ClassSymbol csym = tree.sym; |
14547 | 477 |
//if something went wrong during method applicability check |
478 |
//it is possible that nested expressions inside argument expression |
|
479 |
//are left unchecked - in such cases there's nothing to clean up. |
|
480 |
if (csym == null) return; |
|
25007 | 481 |
typeEnvs.remove(csym); |
36526 | 482 |
chk.removeCompiled(csym); |
33020
7ed4f7ff42c5
8000316: Huge performance bottleneck in com.sun.tools.javac.comp.Check.localClassName
sadayapalam
parents:
32911
diff
changeset
|
483 |
chk.clearLocalClassNameIndexes(csym); |
36526 | 484 |
syms.removeClass(msym, csym.flatname); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
485 |
super.visitClassDef(tree); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
486 |
} |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
487 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
488 |
|
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
489 |
static class DeferredAttrDiagHandler extends Log.DeferredDiagnosticHandler { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
490 |
|
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
491 |
static class PosScanner extends TreeScanner { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
492 |
DiagnosticPosition pos; |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
493 |
boolean found = false; |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
494 |
|
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
495 |
PosScanner(DiagnosticPosition pos) { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
496 |
this.pos = pos; |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
497 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
498 |
|
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
499 |
@Override |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
500 |
public void scan(JCTree tree) { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
501 |
if (tree != null && |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
502 |
tree.pos() == pos) { |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
503 |
found = true; |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
504 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
505 |
super.scan(tree); |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
506 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
507 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
508 |
|
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
509 |
DeferredAttrDiagHandler(Log log, JCTree newTree) { |
42827
36468b5fa7f4
8181370: Convert anonymous inner classes into lambdas/method references
mcimadamore
parents:
38516
diff
changeset
|
510 |
super(log, d -> { |
36468b5fa7f4
8181370: Convert anonymous inner classes into lambdas/method references
mcimadamore
parents:
38516
diff
changeset
|
511 |
PosScanner posScanner = new PosScanner(d.getDiagnosticPosition()); |
36468b5fa7f4
8181370: Convert anonymous inner classes into lambdas/method references
mcimadamore
parents:
38516
diff
changeset
|
512 |
posScanner.scan(newTree); |
36468b5fa7f4
8181370: Convert anonymous inner classes into lambdas/method references
mcimadamore
parents:
38516
diff
changeset
|
513 |
return posScanner.found; |
28142
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
514 |
}); |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
515 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
516 |
} |
32a6b1af81b1
8064365: Better support for finder capabilities in target-typing context
mcimadamore
parents:
27551
diff
changeset
|
517 |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
518 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
519 |
* A deferred context is created on each method check. A deferred context is |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
520 |
* used to keep track of information associated with the method check, such as |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
521 |
* the symbol of the method being checked, the overload resolution phase, |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
522 |
* the kind of attribution mode to be applied to deferred types and so forth. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
523 |
* As deferred types are processed (by the method check routine) stuck AST nodes |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
524 |
* are added (as new deferred attribution nodes) to this context. The complete() |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
525 |
* routine makes sure that all pending nodes are properly processed, by |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
526 |
* progressively instantiating all inference variables on which one or more |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
527 |
* deferred attribution node is stuck. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
528 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
529 |
class DeferredAttrContext { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
530 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
531 |
/** attribution mode */ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
532 |
final AttrMode mode; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
533 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
534 |
/** symbol of the method being checked */ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
535 |
final Symbol msym; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
536 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
537 |
/** method resolution step */ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
538 |
final Resolve.MethodResolutionPhase phase; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
539 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
540 |
/** inference context */ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
541 |
final InferenceContext inferenceContext; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
542 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
543 |
/** parent deferred context */ |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
544 |
final DeferredAttrContext parent; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
545 |
|
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
546 |
/** Warner object to report warnings */ |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
547 |
final Warner warn; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
548 |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
549 |
/** list of deferred attribution nodes to be processed */ |
22163 | 550 |
ArrayList<DeferredAttrNode> deferredAttrNodes = new ArrayList<>(); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
551 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
552 |
DeferredAttrContext(AttrMode mode, Symbol msym, MethodResolutionPhase phase, |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
553 |
InferenceContext inferenceContext, DeferredAttrContext parent, Warner warn) { |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
554 |
this.mode = mode; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
555 |
this.msym = msym; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
556 |
this.phase = phase; |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
557 |
this.parent = parent; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
558 |
this.warn = warn; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
559 |
this.inferenceContext = inferenceContext; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
560 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
561 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
562 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
563 |
* Adds a node to the list of deferred attribution nodes - used by Resolve.rawCheckArgumentsApplicable |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
564 |
* Nodes added this way act as 'roots' for the out-of-order method checking process. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
565 |
*/ |
19914 | 566 |
void addDeferredAttrNode(final DeferredType dt, ResultInfo resultInfo, |
567 |
DeferredStuckPolicy deferredStuckPolicy) { |
|
568 |
deferredAttrNodes.add(new DeferredAttrNode(dt, resultInfo, deferredStuckPolicy)); |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
569 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
570 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
571 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
572 |
* Incrementally process all nodes, by skipping 'stuck' nodes and attributing |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
573 |
* 'unstuck' ones. If at any point no progress can be made (no 'unstuck' nodes) |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
574 |
* some inference variable might get eagerly instantiated so that all nodes |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
575 |
* can be type-checked. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
576 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
577 |
void complete() { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
578 |
while (!deferredAttrNodes.isEmpty()) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
579 |
boolean progress = false; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
580 |
//scan a defensive copy of the node list - this is because a deferred |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
581 |
//attribution round can add new nodes to the list |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
582 |
for (DeferredAttrNode deferredAttrNode : List.from(deferredAttrNodes)) { |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
583 |
if (deferredAttrNode.process(this)) { |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
584 |
deferredAttrNodes.remove(deferredAttrNode); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
585 |
progress = true; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
586 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
587 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
588 |
if (!progress) { |
25444
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
589 |
if (insideOverloadPhase()) { |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
590 |
for (DeferredAttrNode deferredNode: deferredAttrNodes) { |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
591 |
deferredNode.dt.tree.type = Type.noType; |
19914 | 592 |
} |
25444
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
593 |
return; |
19914 | 594 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
595 |
//remove all variables that have already been instantiated |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
596 |
//from the list of stuck variables |
19914 | 597 |
try { |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
598 |
//find stuck expression to unstuck |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
599 |
DeferredAttrNode toUnstuck = pickDeferredNode(); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
600 |
inferenceContext.solveAny(List.from(toUnstuck.deferredStuckPolicy.stuckVars()), warn); |
19914 | 601 |
inferenceContext.notifyChange(); |
602 |
} catch (Infer.GraphStrategy.NodeNotFoundException ex) { |
|
603 |
//this means that we are in speculative mode and the |
|
604 |
//set of contraints are too tight for progess to be made. |
|
605 |
//Just leave the remaining expressions as stuck. |
|
606 |
break; |
|
607 |
} |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
608 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
609 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
610 |
} |
25444
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
611 |
|
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
612 |
public boolean insideOverloadPhase() { |
25444
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
613 |
DeferredAttrContext dac = this; |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
614 |
if (dac == emptyDeferredAttrContext) { |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
615 |
return false; |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
616 |
} |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
617 |
if (dac.mode == AttrMode.SPECULATIVE) { |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
618 |
return true; |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
619 |
} |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
620 |
return dac.parent.insideOverloadPhase(); |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
621 |
} |
38516
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
622 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
623 |
/** |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
624 |
* Pick the deferred node to be unstuck. The chosen node is the first strongly connected |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
625 |
* component containing exactly one node found in the dependency graph induced by deferred nodes. |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
626 |
* If no such component is found, the first deferred node is returned. |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
627 |
*/ |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
628 |
DeferredAttrNode pickDeferredNode() { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
629 |
List<StuckNode> nodes = deferredAttrNodes.stream() |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
630 |
.map(StuckNode::new) |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
631 |
.collect(List.collector()); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
632 |
//init stuck expression graph; a deferred node A depends on a deferred node B iff |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
633 |
//the intersection between A's input variable and B's output variable is non-empty. |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
634 |
for (StuckNode sn1 : nodes) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
635 |
for (Type t : sn1.data.deferredStuckPolicy.stuckVars()) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
636 |
for (StuckNode sn2 : nodes) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
637 |
if (sn1 != sn2 && sn2.data.deferredStuckPolicy.depVars().contains(t)) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
638 |
sn1.deps.add(sn2); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
639 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
640 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
641 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
642 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
643 |
//compute tarjan on the stuck graph |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
644 |
List<? extends StuckNode> csn = GraphUtils.tarjan(nodes).get(0); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
645 |
return csn.length() == 1 ? csn.get(0).data : deferredAttrNodes.get(0); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
646 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
647 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
648 |
class StuckNode extends GraphUtils.TarjanNode<DeferredAttrNode, StuckNode> { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
649 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
650 |
Set<StuckNode> deps = new HashSet<>(); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
651 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
652 |
StuckNode(DeferredAttrNode data) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
653 |
super(data); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
654 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
655 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
656 |
@Override |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
657 |
public DependencyKind[] getSupportedDependencyKinds() { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
658 |
return new DependencyKind[] { Infer.DependencyKind.STUCK }; |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
659 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
660 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
661 |
@Override |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
662 |
public Collection<? extends StuckNode> getDependenciesByKind(DependencyKind dk) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
663 |
if (dk == Infer.DependencyKind.STUCK) { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
664 |
return deps; |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
665 |
} else { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
666 |
throw new IllegalStateException(); |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
667 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
668 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
669 |
|
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
670 |
@Override |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
671 |
public Iterable<? extends StuckNode> getAllDependencies() { |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
672 |
return deps; |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
673 |
} |
b643c42e9d25
8156954: javac incorrectly complains of incompatible types
mcimadamore
parents:
38512
diff
changeset
|
674 |
} |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
675 |
} |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
676 |
|
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
677 |
/** |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
678 |
* Class representing a deferred attribution node. It keeps track of |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
679 |
* a deferred type, along with the expected target type information. |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
680 |
*/ |
19914 | 681 |
class DeferredAttrNode { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
682 |
|
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
683 |
/** underlying deferred type */ |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
684 |
DeferredType dt; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
685 |
|
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
686 |
/** underlying target type information */ |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
687 |
ResultInfo resultInfo; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
688 |
|
19914 | 689 |
/** stuck policy associated with this node */ |
690 |
DeferredStuckPolicy deferredStuckPolicy; |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
691 |
|
19914 | 692 |
DeferredAttrNode(DeferredType dt, ResultInfo resultInfo, DeferredStuckPolicy deferredStuckPolicy) { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
693 |
this.dt = dt; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
694 |
this.resultInfo = resultInfo; |
19914 | 695 |
this.deferredStuckPolicy = deferredStuckPolicy; |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
696 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
697 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
698 |
/** |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
699 |
* Process a deferred attribution node. |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
700 |
* Invariant: a stuck node cannot be processed. |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
701 |
*/ |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
702 |
@SuppressWarnings("fallthrough") |
19914 | 703 |
boolean process(final DeferredAttrContext deferredAttrContext) { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
704 |
switch (deferredAttrContext.mode) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
705 |
case SPECULATIVE: |
19914 | 706 |
if (deferredStuckPolicy.isStuck()) { |
707 |
dt.check(resultInfo, dummyStuckPolicy, new StructuralStuckChecker()); |
|
708 |
return true; |
|
709 |
} else { |
|
710 |
Assert.error("Cannot get here"); |
|
711 |
} |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
712 |
case CHECK: |
19914 | 713 |
if (deferredStuckPolicy.isStuck()) { |
15717 | 714 |
//stuck expression - see if we can propagate |
715 |
if (deferredAttrContext.parent != emptyDeferredAttrContext && |
|
19914 | 716 |
Type.containsAny(deferredAttrContext.parent.inferenceContext.inferencevars, |
717 |
List.from(deferredStuckPolicy.stuckVars()))) { |
|
718 |
deferredAttrContext.parent.addDeferredAttrNode(dt, |
|
719 |
resultInfo.dup(new Check.NestedCheckContext(resultInfo.checkContext) { |
|
720 |
@Override |
|
721 |
public InferenceContext inferenceContext() { |
|
722 |
return deferredAttrContext.parent.inferenceContext; |
|
723 |
} |
|
724 |
@Override |
|
725 |
public DeferredAttrContext deferredAttrContext() { |
|
726 |
return deferredAttrContext.parent; |
|
727 |
} |
|
728 |
}), deferredStuckPolicy); |
|
729 |
dt.tree.type = Type.stuckType; |
|
15717 | 730 |
return true; |
731 |
} else { |
|
732 |
return false; |
|
733 |
} |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
734 |
} else { |
25444
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
735 |
Assert.check(!deferredAttrContext.insideOverloadPhase(), |
27045478cf23
8033483: Should ignore nested lambda bodies during overload resolution
vromero
parents:
25443
diff
changeset
|
736 |
"attribution shouldn't be happening here"); |
19914 | 737 |
ResultInfo instResultInfo = |
738 |
resultInfo.dup(deferredAttrContext.inferenceContext.asInstType(resultInfo.pt)); |
|
739 |
dt.check(instResultInfo, dummyStuckPolicy, basicCompleter); |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
740 |
return true; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
741 |
} |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
742 |
default: |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
743 |
throw new AssertionError("Bad mode"); |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
744 |
} |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
745 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
746 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
747 |
/** |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
748 |
* Structural checker for stuck expressions |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
749 |
*/ |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
750 |
class StructuralStuckChecker extends TreeScanner implements DeferredTypeCompleter { |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
751 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
752 |
ResultInfo resultInfo; |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
753 |
InferenceContext inferenceContext; |
19914 | 754 |
Env<AttrContext> env; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
755 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
756 |
public Type complete(DeferredType dt, ResultInfo resultInfo, DeferredAttrContext deferredAttrContext) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
757 |
this.resultInfo = resultInfo; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
758 |
this.inferenceContext = deferredAttrContext.inferenceContext; |
19914 | 759 |
this.env = dt.env; |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
760 |
dt.tree.accept(this); |
19914 | 761 |
dt.speculativeCache.put(stuckTree, resultInfo); |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
762 |
return Type.noType; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
763 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
764 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
765 |
@Override |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
766 |
public void visitLambda(JCLambda tree) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
767 |
Check.CheckContext checkContext = resultInfo.checkContext; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
768 |
Type pt = resultInfo.pt; |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
769 |
if (!inferenceContext.inferencevars.contains(pt)) { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
770 |
//must be a functional descriptor |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
771 |
Type descriptorType = null; |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
772 |
try { |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
773 |
descriptorType = types.findDescriptorType(pt); |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
774 |
} catch (Types.FunctionDescriptorLookupError ex) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
775 |
checkContext.report(null, ex.getDiagnostic()); |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
776 |
} |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
777 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
778 |
if (descriptorType.getParameterTypes().length() != tree.params.length()) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
779 |
checkContext.report(tree, |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
780 |
diags.fragment("incompatible.arg.types.in.lambda")); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
781 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
782 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
783 |
Type currentReturnType = descriptorType.getReturnType(); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
784 |
boolean returnTypeIsVoid = currentReturnType.hasTag(VOID); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
785 |
if (tree.getBodyKind() == BodyKind.EXPRESSION) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
786 |
boolean isExpressionCompatible = !returnTypeIsVoid || |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
787 |
TreeInfo.isExpressionStatement((JCExpression)tree.getBody()); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
788 |
if (!isExpressionCompatible) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
789 |
resultInfo.checkContext.report(tree.pos(), |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
790 |
diags.fragment("incompatible.ret.type.in.lambda", |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
791 |
diags.fragment("missing.ret.val", currentReturnType))); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
792 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
793 |
} else { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
794 |
LambdaBodyStructChecker lambdaBodyChecker = |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
795 |
new LambdaBodyStructChecker(); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
796 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
797 |
tree.body.accept(lambdaBodyChecker); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
798 |
boolean isVoidCompatible = lambdaBodyChecker.isVoidCompatible; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
799 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
800 |
if (returnTypeIsVoid) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
801 |
if (!isVoidCompatible) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
802 |
resultInfo.checkContext.report(tree.pos(), |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
803 |
diags.fragment("unexpected.ret.val")); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
804 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
805 |
} else { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
806 |
boolean isValueCompatible = lambdaBodyChecker.isPotentiallyValueCompatible |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
807 |
&& !canLambdaBodyCompleteNormally(tree); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
808 |
if (!isValueCompatible && !isVoidCompatible) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
809 |
log.error(tree.body.pos(), |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
810 |
"lambda.body.neither.value.nor.void.compatible"); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
811 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
812 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
813 |
if (!isValueCompatible) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
814 |
resultInfo.checkContext.report(tree.pos(), |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
815 |
diags.fragment("incompatible.ret.type.in.lambda", |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
816 |
diags.fragment("missing.ret.val", currentReturnType))); |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
817 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
818 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
819 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
820 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
821 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
822 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
823 |
boolean canLambdaBodyCompleteNormally(JCLambda tree) { |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
824 |
List<JCVariableDecl> oldParams = tree.params; |
32911
6ee3c12d2d18
8137269: Add better support for local caching in ArgumentAttr
mcimadamore
parents:
32709
diff
changeset
|
825 |
LocalCacheContext localCacheContext = argumentAttr.withLocalCacheContext(); |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
826 |
try { |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
827 |
tree.params = tree.params.stream() |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
828 |
.map(vd -> make.VarDef(vd.mods, vd.name, make.Erroneous(), null)) |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
829 |
.collect(List.collector()); |
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
830 |
return attribSpeculativeLambda(tree, env, attr.unknownExprInfo).canCompleteNormally; |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
831 |
} finally { |
32911
6ee3c12d2d18
8137269: Add better support for local caching in ArgumentAttr
mcimadamore
parents:
32709
diff
changeset
|
832 |
localCacheContext.leave(); |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
833 |
tree.params = oldParams; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
834 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
835 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
836 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
837 |
@Override |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
838 |
public void visitNewClass(JCNewClass tree) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
839 |
//do nothing |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
840 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
841 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
842 |
@Override |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
843 |
public void visitApply(JCMethodInvocation tree) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
844 |
//do nothing |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
845 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
846 |
|
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
847 |
@Override |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
848 |
public void visitReference(JCMemberReference tree) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
849 |
Check.CheckContext checkContext = resultInfo.checkContext; |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
850 |
Type pt = resultInfo.pt; |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
851 |
if (!inferenceContext.inferencevars.contains(pt)) { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
852 |
try { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
853 |
types.findDescriptorType(pt); |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
854 |
} catch (Types.FunctionDescriptorLookupError ex) { |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
855 |
checkContext.report(null, ex.getDiagnostic()); |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
856 |
} |
19914 | 857 |
Env<AttrContext> localEnv = env.dup(tree); |
26532
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
858 |
JCExpression exprTree = (JCExpression)attribSpeculative(tree.getQualifierExpression(), localEnv, |
aa84b6606229
8056021: checkin for JDK-8027262 breaks Checker Framework
jfranck
parents:
26531
diff
changeset
|
859 |
attr.memberReferenceQualifierResult(tree)); |
20249
93f8eae31092
6386236: Please rename com.sun.tools.javac.util.ListBuffer.lb()
alundblad
parents:
19914
diff
changeset
|
860 |
ListBuffer<Type> argtypes = new ListBuffer<>(); |
19914 | 861 |
for (Type t : types.findDescriptorType(pt).getParameterTypes()) { |
862 |
argtypes.append(Type.noType); |
|
863 |
} |
|
864 |
JCMemberReference mref2 = new TreeCopier<Void>(make).copy(tree); |
|
865 |
mref2.expr = exprTree; |
|
21718
74a5882faf79
8026231: Look at 'static' flag when checking method references
vromero
parents:
20249
diff
changeset
|
866 |
Symbol lookupSym = |
28456
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
867 |
rs.resolveMemberReference(localEnv, mref2, exprTree.type, |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
868 |
tree.name, argtypes.toList(), List.nil(), rs.arityMethodCheck, |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
869 |
inferenceContext, rs.structuralReferenceChooser).fst; |
21718
74a5882faf79
8026231: Look at 'static' flag when checking method references
vromero
parents:
20249
diff
changeset
|
870 |
switch (lookupSym.kind) { |
27224
228abfa87080
8054457: Refactor Symbol kinds from small ints to an enum
emc
parents:
26781
diff
changeset
|
871 |
case WRONG_MTH: |
228abfa87080
8054457: Refactor Symbol kinds from small ints to an enum
emc
parents:
26781
diff
changeset
|
872 |
case WRONG_MTHS: |
28456
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
873 |
//note: as argtypes are erroneous types, type-errors must |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
874 |
//have been caused by arity mismatch |
28458
f20c5dd05e34
8069164: Fix langtools make build so that diagnostic framework can be used
mcimadamore
parents:
28457
diff
changeset
|
875 |
checkContext.report(tree, diags.fragment(Fragments.IncompatibleArgTypesInMref)); |
28456
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
876 |
break; |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
877 |
case ABSENT_MTH: |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
878 |
case STATICERR: |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
879 |
//if no method found, or method found with wrong staticness, report better message |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
880 |
checkContext.report(tree, ((ResolveError)lookupSym).getDiagnostic(DiagnosticType.FRAGMENT, |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
881 |
tree, exprTree.type.tsym, exprTree.type, tree.name, argtypes.toList(), List.nil())); |
b87fdde1404d
8068995: Cleanup method reference lookup code
mcimadamore
parents:
28275
diff
changeset
|
882 |
break; |
16294
0c291a3cd60d
8007462: Fix provisional applicability for method references
mcimadamore
parents:
15717
diff
changeset
|
883 |
} |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
884 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
885 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
886 |
} |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
887 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
888 |
/* This visitor looks for return statements, its analysis will determine if |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
889 |
* a lambda body is void or value compatible. We must analyze return |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
890 |
* statements contained in the lambda body only, thus any return statement |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
891 |
* contained in an inner class or inner lambda body, should be ignored. |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
892 |
*/ |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
893 |
class LambdaBodyStructChecker extends TreeScanner { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
894 |
boolean isVoidCompatible = true; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
895 |
boolean isPotentiallyValueCompatible = true; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
896 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
897 |
@Override |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
898 |
public void visitClassDef(JCClassDecl tree) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
899 |
// do nothing |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
900 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
901 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
902 |
@Override |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
903 |
public void visitLambda(JCLambda tree) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
904 |
// do nothing |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
905 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
906 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
907 |
@Override |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
908 |
public void visitNewClass(JCNewClass tree) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
909 |
// do nothing |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
910 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
911 |
|
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
912 |
@Override |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
913 |
public void visitReturn(JCReturn tree) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
914 |
if (tree.expr != null) { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
915 |
isVoidCompatible = false; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
916 |
} else { |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
917 |
isPotentiallyValueCompatible = false; |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
918 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
919 |
} |
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
920 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
921 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
922 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
923 |
/** an empty deferred attribution context - all methods throw exceptions */ |
18910
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
924 |
final DeferredAttrContext emptyDeferredAttrContext; |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
925 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
926 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
927 |
* Map a list of types possibly containing one or more deferred types |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
928 |
* into a list of ordinary types. Each deferred type D is mapped into a type T, |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
929 |
* where T is computed by retrieving the type that has already been |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
930 |
* computed for D during a previous deferred attribution round of the given kind. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
931 |
*/ |
29554
6d7957bd6866
8074100: Turn Type.Mapping into a true visitor
mcimadamore
parents:
29292
diff
changeset
|
932 |
class DeferredTypeMap extends TypeMapping<Void> { |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
933 |
DeferredAttrContext deferredAttrContext; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
934 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
935 |
protected DeferredTypeMap(AttrMode mode, Symbol msym, MethodResolutionPhase phase) { |
15706
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
936 |
this.deferredAttrContext = new DeferredAttrContext(mode, msym, phase, |
37a81a6987ac
8007479: Refactor DeferredAttrContext so that it points to parent context
mcimadamore
parents:
15705
diff
changeset
|
937 |
infer.emptyContext, emptyDeferredAttrContext, types.noWarnings); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
938 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
939 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
940 |
@Override |
29554
6d7957bd6866
8074100: Turn Type.Mapping into a true visitor
mcimadamore
parents:
29292
diff
changeset
|
941 |
public Type visitType(Type t, Void _unused) { |
14359
d4099818ab70
7200915: convert TypeTags from a series of small ints to an enum
jjg
parents:
14258
diff
changeset
|
942 |
if (!t.hasTag(DEFERRED)) { |
29554
6d7957bd6866
8074100: Turn Type.Mapping into a true visitor
mcimadamore
parents:
29292
diff
changeset
|
943 |
return super.visitType(t, null); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
944 |
} else { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
945 |
DeferredType dt = (DeferredType)t; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
946 |
return typeOf(dt); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
947 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
948 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
949 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
950 |
protected Type typeOf(DeferredType dt) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
951 |
switch (deferredAttrContext.mode) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
952 |
case CHECK: |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
953 |
return dt.tree.type == null ? Type.noType : dt.tree.type; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
954 |
case SPECULATIVE: |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
955 |
return dt.speculativeType(deferredAttrContext.msym, deferredAttrContext.phase); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
956 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
957 |
Assert.error(); |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
958 |
return null; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
959 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
960 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
961 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
962 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
963 |
* Specialized recovery deferred mapping. |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
964 |
* Each deferred type D is mapped into a type T, where T is computed either by |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
965 |
* (i) retrieving the type that has already been computed for D during a previous |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
966 |
* attribution round (as before), or (ii) by synthesizing a new type R for D |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
967 |
* (the latter step is useful in a recovery scenario). |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
968 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
969 |
public class RecoveryDeferredTypeMap extends DeferredTypeMap { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
970 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
971 |
public RecoveryDeferredTypeMap(AttrMode mode, Symbol msym, MethodResolutionPhase phase) { |
14547 | 972 |
super(mode, msym, phase != null ? phase : MethodResolutionPhase.BOX); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
973 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
974 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
975 |
@Override |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
976 |
protected Type typeOf(DeferredType dt) { |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
977 |
Type owntype = super.typeOf(dt); |
14547 | 978 |
return owntype == Type.noType ? |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
979 |
recover(dt) : owntype; |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
980 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
981 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
982 |
/** |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
983 |
* Synthesize a type for a deferred type that hasn't been previously |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
984 |
* reduced to an ordinary type. Functional deferred types and conditionals |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
985 |
* are mapped to themselves, in order to have a richer diagnostic |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
986 |
* representation. Remaining deferred types are attributed using |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
987 |
* a default expected type (j.l.Object). |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
988 |
*/ |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
989 |
private Type recover(DeferredType dt) { |
18902 | 990 |
dt.check(attr.new RecoveryInfo(deferredAttrContext) { |
991 |
@Override |
|
992 |
protected Type check(DiagnosticPosition pos, Type found) { |
|
993 |
return chk.checkNonVoid(pos, super.check(pos, found)); |
|
994 |
} |
|
995 |
}); |
|
29554
6d7957bd6866
8074100: Turn Type.Mapping into a true visitor
mcimadamore
parents:
29292
diff
changeset
|
996 |
return super.visit(dt); |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
997 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
998 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
999 |
|
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
1000 |
/** |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1001 |
* A special tree scanner that would only visit portions of a given tree. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1002 |
* The set of nodes visited by the scanner can be customized at construction-time. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1003 |
*/ |
32709
55d136799f79
8078093: Severe compiler performance regression Java 7 to 8 for nested method invocations
mcimadamore
parents:
31937
diff
changeset
|
1004 |
abstract static class FilterScanner extends com.sun.tools.javac.tree.TreeScanner { |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1005 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1006 |
final Filter<JCTree> treeFilter; |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1007 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1008 |
FilterScanner(final Set<JCTree.Tag> validTags) { |
42827
36468b5fa7f4
8181370: Convert anonymous inner classes into lambdas/method references
mcimadamore
parents:
38516
diff
changeset
|
1009 |
this.treeFilter = t -> validTags.contains(t.getTag()); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1010 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1011 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1012 |
@Override |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1013 |
public void scan(JCTree tree) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1014 |
if (tree != null) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1015 |
if (treeFilter.accepts(tree)) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1016 |
super.scan(tree); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1017 |
} else { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1018 |
skip(tree); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1019 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1020 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1021 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1022 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1023 |
/** |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1024 |
* handler that is executed when a node has been discarded |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1025 |
*/ |
24066
1dfb66929538
8029718: Should always use lambda body structure to disambiguate overload resolution
vromero
parents:
22163
diff
changeset
|
1026 |
void skip(JCTree tree) {} |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1027 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1028 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1029 |
/** |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1030 |
* A tree scanner suitable for visiting the target-type dependent nodes of |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1031 |
* a given argument expression. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1032 |
*/ |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1033 |
static class PolyScanner extends FilterScanner { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1034 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1035 |
PolyScanner() { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1036 |
super(EnumSet.of(CONDEXPR, PARENS, LAMBDA, REFERENCE)); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1037 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1038 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1039 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1040 |
/** |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1041 |
* A tree scanner suitable for visiting the target-type dependent nodes nested |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1042 |
* within a lambda expression body. |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1043 |
*/ |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1044 |
static class LambdaReturnScanner extends FilterScanner { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1045 |
|
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1046 |
LambdaReturnScanner() { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1047 |
super(EnumSet.of(BLOCK, CASE, CATCH, DOLOOP, FOREACHLOOP, |
25275
e5e9bf4b8c49
8042759: Lambda returning implicitly-typed lambdas considered pertinent to applicability
vromero
parents:
25007
diff
changeset
|
1048 |
FORLOOP, IF, RETURN, SYNCHRONIZED, SWITCH, TRY, WHILELOOP)); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1049 |
} |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1050 |
} |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1051 |
|
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1052 |
/** |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1053 |
* This visitor is used to check that structural expressions conform |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1054 |
* to their target - this step is required as inference could end up |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1055 |
* inferring types that make some of the nested expressions incompatible |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1056 |
* with their corresponding instantiated target |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1057 |
*/ |
19914 | 1058 |
class CheckStuckPolicy extends PolyScanner implements DeferredStuckPolicy, Infer.FreeTypeListener { |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1059 |
|
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1060 |
Type pt; |
31937
ad43a6639c4a
8132215: class InferenceContext should live in a separate file
vromero
parents:
30728
diff
changeset
|
1061 |
InferenceContext inferenceContext; |
22163 | 1062 |
Set<Type> stuckVars = new LinkedHashSet<>(); |
1063 |
Set<Type> depVars = new LinkedHashSet<>(); |
|
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1064 |
|
19914 | 1065 |
@Override |
1066 |
public boolean isStuck() { |
|
1067 |
return !stuckVars.isEmpty(); |
|
1068 |
} |
|
1069 |
||
1070 |
@Override |
|
1071 |
public Set<Type> stuckVars() { |
|
1072 |
return stuckVars; |
|
1073 |
} |
|
1074 |
||
1075 |
@Override |
|
1076 |
public Set<Type> depVars() { |
|
1077 |
return depVars; |
|
1078 |
} |
|
1079 |
||
1080 |
public CheckStuckPolicy(ResultInfo resultInfo, DeferredType dt) { |
|
1081 |
this.pt = resultInfo.pt; |
|
1082 |
this.inferenceContext = resultInfo.checkContext.inferenceContext(); |
|
1083 |
scan(dt.tree); |
|
1084 |
if (!stuckVars.isEmpty()) { |
|
1085 |
resultInfo.checkContext.inferenceContext() |
|
1086 |
.addFreeTypeListener(List.from(stuckVars), this); |
|
1087 |
} |
|
1088 |
} |
|
1089 |
||
1090 |
@Override |
|
1091 |
public void typesInferred(InferenceContext inferenceContext) { |
|
1092 |
stuckVars.clear(); |
|
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1093 |
} |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1094 |
|
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1095 |
@Override |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1096 |
public void visitLambda(JCLambda tree) { |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1097 |
if (inferenceContext.inferenceVars().contains(pt)) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1098 |
stuckVars.add(pt); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1099 |
} |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
1100 |
if (!types.isFunctionalInterface(pt)) { |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1101 |
return; |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1102 |
} |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1103 |
Type descType = types.findDescriptorType(pt); |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1104 |
List<Type> freeArgVars = inferenceContext.freeVarsIn(descType.getParameterTypes()); |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
1105 |
if (tree.paramKind == JCLambda.ParameterKind.IMPLICIT && |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1106 |
freeArgVars.nonEmpty()) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1107 |
stuckVars.addAll(freeArgVars); |
19914 | 1108 |
depVars.addAll(inferenceContext.freeVarsIn(descType.getReturnType())); |
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1109 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1110 |
scanLambdaBody(tree, descType.getReturnType()); |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1111 |
} |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1112 |
|
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1113 |
@Override |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1114 |
public void visitReference(JCMemberReference tree) { |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1115 |
scan(tree.expr); |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1116 |
if (inferenceContext.inferenceVars().contains(pt)) { |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1117 |
stuckVars.add(pt); |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1118 |
return; |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1119 |
} |
15374
fb8f6acf09cc
8005244: Implement overload resolution as per latest spec EDR
mcimadamore
parents:
15039
diff
changeset
|
1120 |
if (!types.isFunctionalInterface(pt)) { |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1121 |
return; |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1122 |
} |
14547 | 1123 |
|
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1124 |
Type descType = types.findDescriptorType(pt); |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1125 |
List<Type> freeArgVars = inferenceContext.freeVarsIn(descType.getParameterTypes()); |
19914 | 1126 |
if (freeArgVars.nonEmpty() && |
1127 |
tree.overloadKind == JCMemberReference.OverloadKind.OVERLOADED) { |
|
1128 |
stuckVars.addAll(freeArgVars); |
|
1129 |
depVars.addAll(inferenceContext.freeVarsIn(descType.getReturnType())); |
|
18910
c967bfda9283
8016175: Add bottom-up type-checking support for unambiguous method references
mcimadamore
parents:
18902
diff
changeset
|
1130 |
} |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1131 |
} |
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1132 |
|
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1133 |
void scanLambdaBody(JCLambda lambda, final Type pt) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1134 |
if (lambda.getBodyKind() == JCTree.JCLambda.BodyKind.EXPRESSION) { |
19914 | 1135 |
Type prevPt = this.pt; |
1136 |
try { |
|
1137 |
this.pt = pt; |
|
1138 |
scan(lambda.body); |
|
1139 |
} finally { |
|
1140 |
this.pt = prevPt; |
|
1141 |
} |
|
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1142 |
} else { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1143 |
LambdaReturnScanner lambdaScanner = new LambdaReturnScanner() { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1144 |
@Override |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1145 |
public void visitReturn(JCReturn tree) { |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1146 |
if (tree.expr != null) { |
19914 | 1147 |
Type prevPt = CheckStuckPolicy.this.pt; |
1148 |
try { |
|
1149 |
CheckStuckPolicy.this.pt = pt; |
|
1150 |
CheckStuckPolicy.this.scan(tree.expr); |
|
1151 |
} finally { |
|
1152 |
CheckStuckPolicy.this.pt = prevPt; |
|
1153 |
} |
|
15039
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1154 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1155 |
} |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1156 |
}; |
80190ab051c0
8005184: Restructure DeferredAttr to allow pluggable deferred type completers
mcimadamore
parents:
14547
diff
changeset
|
1157 |
lambdaScanner.scan(lambda.body); |
14058
c7ec7facdd20
7177385: Add attribution support for lambda expressions
mcimadamore
parents:
14057
diff
changeset
|
1158 |
} |
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
1159 |
} |
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
1160 |
} |
16975
124c8436d59c
8010923: Avoid redundant speculative attribution
mcimadamore
parents:
16805
diff
changeset
|
1161 |
|
124c8436d59c
8010923: Avoid redundant speculative attribution
mcimadamore
parents:
16805
diff
changeset
|
1162 |
/** |
19914 | 1163 |
* This visitor is used to check that structural expressions conform |
1164 |
* to their target - this step is required as inference could end up |
|
1165 |
* inferring types that make some of the nested expressions incompatible |
|
1166 |
* with their corresponding instantiated target |
|
1167 |
*/ |
|
1168 |
class OverloadStuckPolicy extends CheckStuckPolicy implements DeferredStuckPolicy { |
|
1169 |
||
1170 |
boolean stuck; |
|
1171 |
||
1172 |
@Override |
|
1173 |
public boolean isStuck() { |
|
1174 |
return super.isStuck() || stuck; |
|
1175 |
} |
|
1176 |
||
1177 |
public OverloadStuckPolicy(ResultInfo resultInfo, DeferredType dt) { |
|
1178 |
super(resultInfo, dt); |
|
1179 |
} |
|
1180 |
||
1181 |
@Override |
|
1182 |
public void visitLambda(JCLambda tree) { |
|
1183 |
super.visitLambda(tree); |
|
1184 |
if (tree.paramKind == JCLambda.ParameterKind.IMPLICIT) { |
|
1185 |
stuck = true; |
|
1186 |
} |
|
1187 |
} |
|
1188 |
||
1189 |
@Override |
|
1190 |
public void visitReference(JCMemberReference tree) { |
|
1191 |
super.visitReference(tree); |
|
1192 |
if (tree.overloadKind == JCMemberReference.OverloadKind.OVERLOADED) { |
|
1193 |
stuck = true; |
|
1194 |
} |
|
1195 |
} |
|
1196 |
} |
|
14057
b4b0377b8dba
7177387: Add target-typing support in method context
mcimadamore
parents:
diff
changeset
|
1197 |
} |