<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Nils,<br>
    <br>
    What is a reason for CompilerDirectivesFile option to be diagnostic,
    while CompileCommand is a product one?<br>
    I think they both should be product options.<br>
    <br>
    In src/share/vm/runtime/globals.hpp <br>
    3552   diagnostic(ccstr, CompilerDirectivesFile,
    NULL,                           \<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 07.10.2015 13:38, Nils Eliasson
      wrote:<br>
    </div>
    <blockquote cite="mid:5614F60D.3040602@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      Hi all,<br>
      <br>
      Latest webrev including fixes after comments from Chris and
      Zoltan: <br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Eneliasso/8046155/webrev.09/">http://cr.openjdk.java.net/~neliasso/8046155/webrev.09/</a><br>
      <br>
      Regards,<br>
      //Nils<br>
      <br>
      <br>
      <br>
      On 2015-09-22 19:21, Nils Eliasson wrote:<br>
      <span style="white-space: pre;">> Hi,
        > > This is the initial RFR for JEP165: Compiler Control.
        This feature
        > enables runtime manageable, method dependent compiler
        flags.
        > (Immutable for the duration of a compilation.)
        > > The change includes: - A parser for the directives
        format (json like)
        > including vmtests (json.cpp/hpp) - A component for
        construction of
        > compiler directives (directivesParser.cpp/hpp) - The
        directives
        > including the option definitions, default values and
        compilecommand
        > relations (compilerDirectives.cpp/hpp) - Diagnostic
        commands for
        > working with the directives - installing, removing,
        printing - Lots
        > of small changes wherever we access flags or legacy
        compilecommands
        > in the compiler
        > > Notes: The feature is documented in the JEP
        > (<a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8046155">https://bugs.openjdk.java.net/browse/JDK-8046155</a>).
>
        > Currently only a small amount of compiler flags are
        included in the
        > change. The flags are a representative selection of
        different types
        > targeting both compilers. All of them existed as
        CompilerOracle
        > option commands. Two commands was not included in the
        directives due
        > to time constraints - CompilerThresholdScaling and
        UseRTMLocks. Both
        > are accessed from runtime (outside any compiler) and
        requires some
        > special handling. (Solved but not implemented.)
        > > Full backwards compatibility with CompileCommands is
        implemented but
        > can be turned off with flag -XX:CompileCommandCompatibilty.
        Also meta
        > handling the compatibility flag by supporting it in the
        directives
        > (test feature).
        > > The change contain some rough edges that will polished
        over the
        > coming days.
        > > JEP: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8046155">https://bugs.openjdk.java.net/browse/JDK-8046155</a>
        Hotspot webrev:
        > <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eneliasso/8046155/webrev.01/">http://cr.openjdk.java.net/~neliasso/8046155/webrev.01/</a>
        JDK webrev:
        > <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eneliasso/8046155/webrev_jdk.01/">http://cr.openjdk.java.net/~neliasso/8046155/webrev_jdk.01/</a>
        > > Please review!
        > > Best regards, Nils Eliasson</span><br>
      <br>
      <br>
    </blockquote>
    <br>
    <pre class="moz-signature" cols="72">-- 
Thanks,
Pavel Punegov</pre>
  </body>
</html>