Integrated: 8256508: Improve CompileCommand flag

[Nils Eliasson]

On Tue, 17 Nov 2020 21:41:23 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> The current implementation of compile command has two types of options. Types pre-defined options like "compileonly" and the general 'option' type.
> 
> 'option'-type are not defined, they can accidentally be used with the wrong value type, and the syntax is prone to error.
> 
> By pre-defining all compile commands used and giving them types the parsing can be simplified, proper parsing errors can be given and and reasonable syntax can be used.
> 
> This:
> -XX:CompileCommand=option,java/util/String.toString,int,RepeatCompilation,5
> 
> Is superseded by:
> -XX:CompileCommand=RepeatCompilation,java/util/String.toString,5
> 
> Attention check: Did you spot the error in the old command?
> 
> In order not to break anything - the old syntax is kept for now. But even the old command format is improved with verification for the option name and the type of the value.

This pull request has now been integrated.

Changeset: cfb175df
Author:    Nils Eliasson <neliasso at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/cfb175df
Stats:     945 lines in 22 files changed: 465 ins; 151 del; 329 mod

8256508: Improve CompileCommand flag

Reviewed-by: redestad, kvn

-------------

PR: https://git.openjdk.java.net/jdk/pull/1276