-XD-relax doesn't work

[Maurizio Cimadamore]

On 20/01/16 01:31, Liam Miller-Cushon wrote:
> Thanks, I'll avoid using it :)
>
Meta-question - what were you using it for?

Maurizio
> On Tue, Jan 19, 2016 at 5:03 PM, Maurizio Cimadamore 
> <maurizio.cimadamore at oracle.com 
> <mailto:maurizio.cimadamore at oracle.com>> wrote:
>
>     Thanks
>     I'll add that to the list of flags to cleanup :-)
>
>     Maurizio
>
>
>     On 20/01/16 00:49, Liam Miller-Cushon wrote:
>
>         I think that -XD-relax has been broken since JDK8. My
>         understanding is that it was added in the jsr14 era, so maybe
>         it should just be removed?
>
>         The fix is:
>
>         diff -r 6149fc30cd71
>         src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavaCompiler.java
>         ---
>         a/src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavaCompiler.java
>         Thu Jan 14 12:03:24 2016 -0800
>         +++
>         b/src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavaCompiler.java
>         Tue Jan 19 16:34:56 2016 -0800
>         @@ -1281,6 +1281,7 @@
>                          return;
>
>                      if (relax) {
>         +                compileStates.put(env, CompileState.FLOW);
>                          results.add(env);
>                          return;
>                      }
>
>         Repro:
>
>         $ cat Test.java
>         class Test {}
>         $ javac -XD-relax Test.java
>         An exception has occurred in the compiler (1.9.0-internal).
>         Please file a bug against the Java compiler via the Java bug
>         reporting page (http://bugreport.java.com) after checking the
>         Bug Database (http://bugs.java.com) for duplicates. Include
>         your program and the following diagnostic in your report.
>         Thank you.
>         java.lang.AssertionError: The current compile state [ATTR] of
>         class Test is previous to FLOW
>                 at com.sun.tools.javac.util.Assert.error(Assert.java:162)
>                 at
>         com.sun.tools.javac.comp.TransTypes.translateClass(TransTypes.java:937)
>                 at
>         com.sun.tools.javac.comp.TransTypes.visitClassDef(TransTypes.java:512)
>                 at
>         com.sun.tools.javac.tree.JCTree$JCClassDecl.accept(JCTree.java:744)
>                 at
>         com.sun.tools.javac.tree.TreeTranslator.translate(TreeTranslator.java:58)
>                 at
>         com.sun.tools.javac.comp.TransTypes.translate(TransTypes.java:491)
>                 at
>         com.sun.tools.javac.comp.TransTypes.translateTopLevelClass(TransTypes.java:977)
>                 at
>         com.sun.tools.javac.main.JavaCompiler.desugar(JavaCompiler.java:1455)
>                 at
>         com.sun.tools.javac.main.JavaCompiler.desugar(JavaCompiler.java:1326)
>                 at
>         com.sun.tools.javac.main.JavaCompiler.compile(JavaCompiler.java:871)
>                 at com.sun.tools.javac.main.Main.compile(Main.java:257)
>                 at com.sun.tools.javac.main.Main.compile(Main.java:142)
>                 at com.sun.tools.javac.Main.compile(Main.java:55)
>                 at com.sun.tools.javac.Main.main(Main.java:41)
>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20160120/bf67f557/attachment-0001.html>