Default to averaging for all polybench jobs if no aggregation is specified

farquet · farquet · commit 81e450d85449 · 2025-10-03T16:04:46.000+02:00
diff --git a/truffle/src/org.graalvm.polybench/src/org/graalvm/polybench/Config.java b/truffle/src/org.graalvm.polybench/src/org/graalvm/polybench/Config.java
@@ -140,7 +140,8 @@ private Summary parseFallbackBenchSpecificSummary(Value benchmark) {
         } else if (benchmark.hasMember(AverageSummary.class.getSimpleName())) {
             return new AverageSummary();
         }
-        return null;
+        // defaulting to averaging if the aggregation strategy is not specified
+        return new AverageSummary();
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,8 @@ private Summary parseFallbackBenchSpecificSummary(Value benchmark) {`
`140`	`140`	`} else if (benchmark.hasMember(AverageSummary.class.getSimpleName())) {`
`141`	`141`	`return new AverageSummary();`
`142`	`142`	`}`
`143`		`- return null;`
	`143`	`+ // defaulting to averaging if the aggregation strategy is not specified`
	`144`	`+ return new AverageSummary();`
`144`	`145`	`}`
`145`	`146`
`146`	`147`	`@Override`