Skip to content

Commit 1728631

Browse files
committed
[GR-70747] Relax parsing to handle interleaved output in dacapo results.
PullRequest: graal/22453
2 parents 5bfb6cf + 74c7db4 commit 1728631

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

sdk/mx.sdk/mx_sdk_benchmark.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2231,15 +2231,15 @@ def successPatterns(self):
22312231
return [
22322232
# Due to the non-determinism of DaCapo version printing, we only match the name.
22332233
re.compile(
2234-
r"^===== DaCapo (?P<version>[^\n]+) ([a-zA-Z0-9_]+) PASSED in ([0-9]+) msec =====", # pylint: disable=line-too-long
2234+
r"^===== DaCapo (?P<version>[^\r\n]+) ([a-zA-Z0-9_]+) PASSED in ([0-9]+) msec =====", # pylint: disable=line-too-long
22352235
re.MULTILINE)
22362236
]
22372237

22382238
def failurePatterns(self):
22392239
return [
22402240
# Due to the non-determinism of DaCapo version printing, we only match the name.
22412241
re.compile(
2242-
r"^===== DaCapo (?P<version>[^\n]+) ([a-zA-Z0-9_]+) FAILED (warmup|) =====", # pylint: disable=line-too-long
2242+
r"^===== DaCapo (?P<version>[^\r\n]+) ([a-zA-Z0-9_]+) FAILED (warmup|) =====", # pylint: disable=line-too-long
22432243
re.MULTILINE),
22442244
re.compile(
22452245
r"^\[\[\[Graal compilation failure\]\]\]", # pylint: disable=line-too-long
@@ -2256,7 +2256,7 @@ def rules(self, out, benchmarks, bmSuiteArgs):
22562256
return [
22572257
# Due to the non-determinism of DaCapo version printing, we only match the name.
22582258
mx_benchmark.StdOutRule(
2259-
r"===== DaCapo (?P<version>[^\n]+) (?P<benchmark>[a-zA-Z0-9_]+) PASSED in (?P<time>[0-9]+) msec =====", # pylint: disable=line-too-long
2259+
r"===== DaCapo (?P<version>[^\r\n]+) (?P<benchmark>[a-zA-Z0-9_]+) PASSED in (?P<time>[0-9]+) msec =====", # pylint: disable=line-too-long
22602260
{
22612261
"benchmark": ("<benchmark>", str),
22622262
"bench-suite": self.benchSuiteName(),
@@ -2275,7 +2275,7 @@ def rules(self, out, benchmarks, bmSuiteArgs):
22752275
# The warmup metric should capture all warmup iterations (which print 'completed warmup X') in addition to
22762276
# the last final iteration (which prints 'PASSED').
22772277
mx_benchmark.StdOutRule(
2278-
r"===== DaCapo (?P<version>[^\n]+) (?P<benchmark>[a-zA-Z0-9_]+) ((completed warmup [0-9]+)|PASSED) in (?P<time>[0-9]+) msec =====", # pylint: disable=line-too-long
2278+
r"===== DaCapo (?P<version>[^\r\n]+) (?P<benchmark>[a-zA-Z0-9_]+) ((completed warmup [0-9]+)|PASSED) in (?P<time>[0-9]+) msec =====", # pylint: disable=line-too-long
22792279
{
22802280
"benchmark": ("<benchmark>", str),
22812281
"bench-suite": self.benchSuiteName(),

0 commit comments

Comments
 (0)