8147018: CompilerControl: Improve handling of timeouts and failures for tests

Dump expected method states, improve compile commands dumping in CompilerControl tests

Reviewed-by: iignatyev, rbackman
This commit is contained in:
Evgeny Nikitin 2020-04-23 16:20:17 +02:00
parent fc842d2b4b
commit 91220287fc
10 changed files with 42 additions and 16 deletions

View File

@ -63,7 +63,6 @@ public class AddAndRemoveTest extends AbstractTestBase {
MethodDescriptor md = getValidMethodDescriptor(exec);
CompileCommand compileCommand = new JcmdCommand(Command.COMPILEONLY,
md, null, Scenario.Type.JCMD, Scenario.JcmdType.ADD);
compileCommand.print();
builder.add(compileCommand);
}
// Remove half of them

View File

@ -76,7 +76,6 @@ public class ClearDirectivesFileStackTest extends AbstractTestBase {
CompileCommand compileCommand = new CompileCommand(command,
methodDescriptor, cmdGen.generateCompiler(),
Scenario.Type.DIRECTIVE);
compileCommand.print();
builder.add(compileCommand);
}
// clear the stack

View File

@ -66,7 +66,6 @@ public class ClearDirectivesStackTest extends AbstractTestBase {
cmdGen.generateCommand(), methodDescriptor,
cmdGen.generateCompiler(), Scenario.Type.JCMD,
Scenario.JcmdType.ADD);
compileCommand.print();
builder.add(compileCommand);
}
// clear the stack

View File

@ -73,7 +73,6 @@ public class PrintDirectivesTest extends AbstractTestBase {
CompileCommand compileCommand = new CompileCommand(command,
methodDescriptor, cmdGen.generateCompiler(),
Scenario.Type.DIRECTIVE);
compileCommand.print();
builder.add(compileCommand);
}
// print all directives

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -76,7 +76,6 @@ public class MultiCommand extends AbstractTestBase {
builder.addFlag("-XX:+UnlockDiagnosticVMOptions");
builder.addFlag("-XX:CompilerDirectivesLimit=101");
for (CompileCommand cc : testCases) {
cc.print();
builder.add(cc);
}
Scenario scenario = builder.build();

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -49,7 +49,6 @@ public class SingleCommand extends AbstractTestBase {
CommandGenerator cmdGen = new CommandGenerator();
CompileCommand compileCommand = cmdGen.generateCompileCommand(command,
md, type);
compileCommand.print();
builder.add(compileCommand);
Scenario scenario = builder.build();
scenario.execute();

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -58,6 +58,13 @@ public class CompileAction {
*/
public static void checkCompiled(Executable executable,
State state) {
{ // Dumping the state being checked
System.out.println("Checking expected compilation state: {");
System.out.println(" method: " + executable);
state.toString().lines()
.map(line -> " " + line).forEach(System.out::println);
System.out.println("}");
}
int first = COMP_LEVELS[0];
if (first < 4) {
checkCompilation(executable, first, state.isC1Compilable());

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -57,10 +57,25 @@ public class CompileCommand {
}
/**
* Prints compile command to the system output
* Formats the command according to the following pattern:
* {@code <command_name> Type: <type> Compiler: <compiler> MethodDescriptor: <method_descriptor> IsValid: <true/false>}
* Sample output:
* COMPILEONLY Type: OPTION Compiler: C1 MethodDescriptor: *Klass.method* IsValid: true
*/
public void print() {
System.out.printf("%s (type: %s): %s (valid: %b)%n", command.name(),
type.name(), methodDescriptor.getString(), isValid());
protected String formatFields() {
return command.name() +
" Type: " + type +
" Compiler: " + compiler +
" MethodDescriptor: " + (methodDescriptor == null ? "null" : methodDescriptor.getString()) +
" IsValid: " + isValid();
}
/**
* Returns formatted string representation in the form
* {@code "(CompileCommand Field1: <field1> Field2: <field2> ...)}
* The fields are formatted by {@link #formatFields()}.
*/
public String toString() {
return "(CompileCommand " + formatFields() + ")";
}
}

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -36,4 +36,13 @@ public class JcmdCommand extends CompileCommand {
super(command, methodDescriptor, compiler, type);
this.jcmdType = jcmdType;
}
/**
* Enchances parent's class method with the the JCMDtype printing:
* {@code ... JCMDType: <jcmd_type>}
*/
protected String formatFields() {
return super.formatFields() + " JCMDType: " + jcmdType;
}
}

View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2016, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2020, 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
@ -214,6 +214,7 @@ public final class Scenario {
}
public void add(CompileCommand compileCommand) {
System.out.println(compileCommand);
String[] vmOptions = compileCommand.command.vmOpts;
Collections.addAll(vmopts, vmOptions);
if (compileCommand.command == Command.LOG) {