8008723: Graph Inference: bad graph calculation leads to assertion error
Summary: Dependencies are not propagated correctly through merged nodes during inference graph initialization
Reviewed-by: jjg
--- a/langtools/src/share/classes/com/sun/tools/javac/comp/Infer.java Tue Feb 26 09:04:19 2013 +0000
+++ b/langtools/src/share/classes/com/sun/tools/javac/comp/Infer.java Thu Feb 28 14:00:52 2013 +0000
@@ -1216,7 +1216,7 @@
* created, effectively replacing the original cyclic nodes.
*/
void initNodes() {
- ArrayList<Node> nodes = new ArrayList<Node>();
+ nodes = new ArrayList<Node>();
for (Type t : inferenceContext.restvars()) {
nodes.add(new Node(t));
}
@@ -1235,7 +1235,7 @@
}
}
}
- this.nodes = new ArrayList<Node>();
+ ArrayList<Node> acyclicNodes = new ArrayList<Node>();
for (List<? extends Node> conSubGraph : GraphUtils.tarjan(nodes)) {
if (conSubGraph.length() > 1) {
Node root = conSubGraph.head;
@@ -1244,8 +1244,9 @@
notifyUpdate(n, root);
}
}
- this.nodes.add(conSubGraph.head);
+ acyclicNodes.add(conSubGraph.head);
}
+ nodes = acyclicNodes;
}
/**
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/langtools/test/tools/javac/lambda/TargetType65.java Thu Feb 28 14:00:52 2013 +0000
@@ -0,0 +1,58 @@
+/*
+ * Copyright (c) 2013, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+/*
+ * @test
+ * @bug 8008723
+ * @summary Graph Inference: bad graph calculation leads to assertion error
+ * @compile TargetType65.java
+ */
+class TargetType65 {
+ interface Predicate<X> {
+ boolean accepts(X x);
+ }
+
+ static class Optional<T> {
+ public boolean isPresent() { return false; }
+ public static<E> Optional<E> empty() { return null; }
+ }
+
+ interface Supplier<X> {
+ X make();
+ }
+
+ static class Sink<O, T> { }
+
+ static class SubSink<R> extends Sink<R, Optional<R>> { }
+
+ static class Tester<T, O> {
+ public static <F> Tester<F, Optional<F>> makeRef() {
+ return new Tester<>(Optional.empty(), Optional::isPresent, SubSink::new);
+ }
+
+ private Tester(O emptyValue,
+ Predicate<O> presentPredicate,
+ Supplier<Sink<T, O>> sinkSupplier) {
+ }
+ }
+}