Skip to content

Commit

Permalink
[GR-29961] Improve compiler code coverage.
Browse files Browse the repository at this point in the history
PullRequest: graal/8661
  • Loading branch information
andrewcraik committed Apr 12, 2021
2 parents 836d28b + f8c003a commit 145c843
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
4 changes: 2 additions & 2 deletions compiler/ci_common/gate.hocon
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ gateCoverage : {
EXTRA_VM_ARGS : ""
}
run : [
${gateCmd} ["build,coverage", --jacocout, html]
${gateCmd} ["build,coverage", --jacoco-omit-excluded, --jacocout, html]
[mx, coverage-upload]
# GR-18258 [mx, sonarqube-upload, "-Dsonar.host.url=$SONAR_HOST_URL", "-Dsonar.projectKey=com.oracle.graal.compiler."${jvm-config.default}, "-Dsonar.projectName=GraalVM - Compiler ("${jvm-config.default}")", --exclude-generated, --skip-coverage]
]
Expand Down Expand Up @@ -82,7 +82,7 @@ gateTestCTWWeekly : {
run : [
# At least one gate build should run without strict
# compliance as that's how Travis runs on JDK9
["mx"] ${gateCmdSuffix} ["build,ctw", "--jacocout", "html"]
["mx"] ${gateCmdSuffix} ["build,ctw", "--jacoco-omit-excluded", "--jacocout", "html"]
["mx", "coverage-upload"]
]
timelimit : "1:30:00"
Expand Down
6 changes: 4 additions & 2 deletions compiler/ci_common/gate_tasks.hocon
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@ builds += [
${gateMathStubsListener} {name: "daily-hotspot-mathstubs-listener"}

# Linux AArch64
${gateTest} ${labsjdk-ce-11} ${gateLinuxAArch64} {name: "gate-compiler-test-labsjdk-ce-11-linux-aarch64", timelimit: "1:50:00"}
${gateTestCTW} ${labsjdk-ce-11} ${gateLinuxAArch64} {name: "gate-compiler-ctw-labsjdk-ce-11-linux-aarch64", timelimit: "1:50:00"}
${gateTest} ${labsjdk-ce-11} ${gateLinuxAArch64} {name: "gate-compiler-test-labsjdk-ce-11-linux-aarch64", timelimit: "1:50:00"}
${gateTestCTW} ${labsjdk-ce-11} ${gateLinuxAArch64} {name: "gate-compiler-ctw-labsjdk-ce-11-linux-aarch64", timelimit: "1:50:00"}
${gateCoverage} ${labsjdk-ce-11} ${gateLinuxAArch64} {name: "weekly-compiler-coverage-11-linux-aarch64"} ${graalWeekly} {timelimit: "1:50:00"}
${gateTestCTWWeekly} ${labsjdk-ee-11} ${gateLinuxAArch64} {name: "weekly-test-compiler-ctw-labsjdk-ee-11-linux-aarch64"} ${graalWeekly}
]
2 changes: 1 addition & 1 deletion compiler/mx.compiler/mx_compiler.py
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ def source_supplier():
_check_jvmci_version(jdk)

mx_gate.add_jacoco_includes(['org.graalvm.*'])
mx_gate.add_jacoco_excludes(['com.oracle.truffle.*'])
mx_gate.add_jacoco_excludes(['com.oracle.truffle'])
mx_gate.add_jacoco_excluded_annotations(['@Snippet', '@ClassSubstitution'])

def _get_XX_option_value(vmargs, name, default):
Expand Down

0 comments on commit 145c843

Please sign in to comment.