Skip to content

Commit

Permalink
Remove failure on warnings for compiler gates (compiler-test, ctw, bo…
Browse files Browse the repository at this point in the history 
…otstrap)
  • Loading branch information
@imanemamri
imanemamri committed Mar 8, 2022
1 parent b0e9241 commit cea10ac
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 22 deletions.
2 changes: 2 additions & 0 deletions common.jsonnet
View file
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,8 @@
windows_server_2016_amd64:: self.windows_server_2016 + self.amd64,
linux_aarch64:: self.linux + self.aarch64,

mach5_target:: {targets+: ["mach5"]},

// Utils
disable_proxies:: {
setup+: [["unset", "HTTP_PROXY", "HTTPS_PROXY", "FTP_PROXY", "NO_PROXY", "http_proxy", "https_proxy", "ftp_proxy", "no_proxy"]],
Expand Down
38 changes: 20 additions & 18 deletions compiler/ci_common/gate.jsonnet
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
]
},

base(tags="build,test", cmd_suffix=[], extra_vm_args="", extra_unittest_args="", jvm_config_suffix=null):: s.setup + {
base(tags="build,test", cmd_suffix=[], extra_vm_args="", extra_unittest_args="", jvm_config_suffix=null, no_warning_as_error=false):: s.setup + {
run+: [
["mx", "--strict-compliance",
"--kill-with-sigquit",
Expand All @@ -22,6 +22,8 @@
(if extra_vm_args == "" then "" else " " + extra_vm_args)
] + (if extra_unittest_args != "" then [
"--extra-unittest-argument=" + extra_unittest_args,
] else []) + (if no_warning_as_error then [
"--no-warning-as-error"
] else []) + [
"--tags=" + tags
] + cmd_suffix
Expand All @@ -47,7 +49,7 @@
},


test:: s.base(),
test:: s.base(no_warning_as_error=true),

coverage:: s.base("build,coverage", ["--jacoco-omit-excluded", "--jacocout", "html"]) + {
run+: [
Expand All @@ -71,7 +73,7 @@
"-Dtck.inlineVerifierInstrument=false",
extra_unittest_args="truffle"),

ctw:: s.base("build,ctw"),
ctw:: s.base("build,ctw", no_warning_as_error=true),

ctw_economy:: s.base("build,ctweconomy", extra_vm_args="-Dgraal.CompilerConfiguration=economy"),

Expand Down Expand Up @@ -104,10 +106,10 @@
]
},

bootstrap:: s.base("build,bootstrap"),
bootstrap_lite:: s.base("build,bootstraplite"),
bootstrap_full:: s.base("build,bootstrapfullverify"),
bootstrap_economy:: s.base("build,bootstrapeconomy", extra_vm_args="-Dgraal.CompilerConfiguration=economy"),
bootstrap:: s.base("build,bootstrap", no_warning_as_error=true),
bootstrap_lite:: s.base("build,bootstraplite", no_warning_as_error=true),
bootstrap_full:: s.base("build,bootstrapfullverify", no_warning_as_error=true),
bootstrap_economy:: s.base("build,bootstrapeconomy", no_warning_as_error=true, extra_vm_args="-Dgraal.CompilerConfiguration=economy"),

style:: c.eclipse + c.jdt + s.base("style,fullbuild,javadoc"),

Expand All @@ -124,18 +126,18 @@

builds: [
# Darwin AMD64
{name: "gate-compiler-test-labsjdk-17-darwin-amd64"} + s.test + c.labsjdk17 + c.darwin_amd64 + t("1:00:00") + s.save_as_json,
{name: "gate-compiler-test-labsjdk-17-darwin-amd64"} + s.test + c.labsjdk17 + c.darwin_amd64 + t("1:00:00") + s.save_as_json + c.mach5_target,
{name: "weekly-compiler-test-labsjdk-11-darwin-amd64"} + s.test + c.labsjdk11 + c.darwin_amd64 + s.weekly,

# Windows AMD64
{name: "gate-compiler-test-labsjdk-11-windows-amd64"} + s.test + c.labsjdk11 + c.windows_amd64 + t("55:00") + c.devkits["windows-jdk11"] + s.save_as_json,
{name: "gate-compiler-test-labsjdk-17-windows-amd64"} + s.test + c.labsjdk17 + c.windows_amd64 + t("55:00") + c.devkits["windows-jdk17"] + s.save_as_json,
{name: "gate-compiler-test-labsjdk-11-windows-amd64"} + s.test + c.labsjdk11 + c.windows_amd64 + t("55:00") + c.devkits["windows-jdk11"] + s.save_as_json + c.mach5_target,
{name: "gate-compiler-test-labsjdk-17-windows-amd64"} + s.test + c.labsjdk17 + c.windows_amd64 + t("55:00") + c.devkits["windows-jdk17"] + s.save_as_json + c.mach5_target,

# Linux AMD64
{name: "gate-compiler-test-labsjdk-11-linux-amd64"} + s.test + c.labsjdk11 + c.linux_amd64 + t("50:00") + s.save_as_json,
{name: "gate-compiler-test-labsjdk-17-linux-amd64"} + s.test + c.labsjdk17 + c.linux_amd64 + t("55:00") + s.save_as_json,
{name: "gate-compiler-ctw-labsjdk-11-linux-amd64"} + s.ctw + c.labsjdk11 + c.linux_amd64,
{name: "gate-compiler-ctw-labsjdk-17-linux-amd64"} + s.ctw + c.labsjdk17 + c.linux_amd64,
{name: "gate-compiler-test-labsjdk-11-linux-amd64"} + s.test + c.labsjdk11 + c.linux_amd64 + t("50:00") + s.save_as_json + c.mach5_target,
{name: "gate-compiler-test-labsjdk-17-linux-amd64"} + s.test + c.labsjdk17 + c.linux_amd64 + t("55:00") + s.save_as_json + c.mach5_target,
{name: "gate-compiler-ctw-labsjdk-11-linux-amd64"} + s.ctw + c.labsjdk11 + c.linux_amd64 + c.mach5_target,
{name: "gate-compiler-ctw-labsjdk-17-linux-amd64"} + s.ctw + c.labsjdk17 + c.linux_amd64 + c.mach5_target,
{name: "gate-compiler-ctw-economy-labsjdk-11-linux-amd64"} + s.ctw_economy + c.labsjdk11 + c.linux_amd64,
{name: "gate-compiler-ctw-economy-labsjdk-17-linux-amd64"} + s.ctw_economy + c.labsjdk17 + c.linux_amd64,
{name: "gate-compiler-benchmarktest-labsjdk-11-linux-amd64"} + s.benchmark + c.labsjdk11 + c.linux_amd64,
Expand All @@ -158,11 +160,11 @@
{name: "gate-compiler-ctw-economy-labsjdk-11-linux-aarch64"} + s.ctw_economy + c.labsjdk11 + c.linux_aarch64 + t("1:50:00"),
{name: "weekly-compiler-coverage-labsjdk-11-linux-aarch64"} + s.coverage + c.labsjdk11 + c.linux_aarch64 + s.weekly + t("1:50:00"),
{name: "weekly-compiler-test-ctw-labsjdk-11-linux-aarch64"} + s.coverage_ctw + c.labsjdk11 + c.linux_aarch64 + s.weekly,

# Bootstrap testing
{name: "gate-compiler-bootstraplite-labsjdk-11-darwin-amd64"} + s.bootstrap_lite + c.labsjdk11 + c.darwin_amd64 + t("1:00:00"),
{name: "gate-compiler-bootstraplite-labsjdk-17-darwin-amd64"} + s.bootstrap_lite + c.labsjdk17 + c.darwin_amd64 + t("1:00:00"),
{name: "gate-compiler-bootstrapfullverify-labsjdk-17-linux-amd64"} +s.bootstrap_full + c.labsjdk17 + c.linux_amd64 + s.many_cores,
{name: "gate-compiler-bootstraplite-labsjdk-11-darwin-amd64"} + s.bootstrap_lite + c.labsjdk11 + c.darwin_amd64 + t("1:00:00") + c.mach5_target,
{name: "gate-compiler-bootstraplite-labsjdk-17-darwin-amd64"} + s.bootstrap_lite + c.labsjdk17 + c.darwin_amd64 + t("1:00:00") + c.mach5_target,
{name: "gate-compiler-bootstrapfullverify-labsjdk-17-linux-amd64"} +s.bootstrap_full + c.labsjdk17 + c.linux_amd64 + s.many_cores + c.mach5_target,

] + (import '../ci_includes/bootstrap_extra.libsonnet').builds
}
8 changes: 4 additions & 4 deletions compiler/ci_includes/bootstrap_extra.libsonnet
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@
local g = import '../ci_common/gate.jsonnet',

builds: [
{name: "gate-compiler-bootstrap-labsjdk-ee-11-linux-amd64"} + g.bootstrap + c.labsjdk11 + c.linux_amd64 + g.many_cores,
{name: "gate-compiler-bootstrap-labsjdk-ee-17-linux-amd64"} + g.bootstrap + c.labsjdk17 + c.linux_amd64 + g.many_cores,
{name: "gate-compiler-bootstrap-economy-labsjdk-ee-11-linux-amd64"} + g.bootstrap_economy + c.labsjdk11 + c.linux_amd64 + g.many_cores,
{name: "gate-compiler-bootstrap-economy-labsjdk-ee-17-linux-amd64"} + g.bootstrap_economy + c.labsjdk17 + c.linux_amd64 + g.many_cores,
{name: "gate-compiler-bootstrap-labsjdk-ee-11-linux-amd64"} + g.bootstrap + c.labsjdk11 + c.linux_amd64 + g.many_cores + c.mach5_target,
{name: "gate-compiler-bootstrap-labsjdk-ee-17-linux-amd64"} + g.bootstrap + c.labsjdk17 + c.linux_amd64 + g.many_cores + c.mach5_target,
{name: "gate-compiler-bootstrap-economy-labsjdk-ee-11-linux-amd64"} + g.bootstrap_economy + c.labsjdk11 + c.linux_amd64 + g.many_cores + c.mach5_target,
{name: "gate-compiler-bootstrap-economy-labsjdk-ee-17-linux-amd64"} + g.bootstrap_economy + c.labsjdk17 + c.linux_amd64 + g.many_cores + c.mach5_target,
]
}

0 comments on commit cea10ac

Please sign in to comment.