diff options
author | Laurent Alfonsi <laurent.alfonsi@linaro.org> | 2023-06-22 09:30:38 +0200 |
---|---|---|
committer | Laurent Alfonsi <laurent.alfonsi@linaro.org> | 2023-06-22 18:57:46 +0000 |
commit | 45c0646915879740a9f834b13345ae685595a703 (patch) | |
tree | 9298c2c94058d4e2f9d8929655c6585786102622 /tcwg_bmk-build.sh | |
parent | 4b36216d70e60de56e321d0b403020d27c2d44fe (diff) |
tcwg_bmk-build.sh: Simplify compare_options handling
Change-Id: I4baf0e2f7f900383af51a70d5954b27ab1bdc969
Diffstat (limited to 'tcwg_bmk-build.sh')
-rwxr-xr-x | tcwg_bmk-build.sh | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/tcwg_bmk-build.sh b/tcwg_bmk-build.sh index 291abea2..4d89c66b 100755 --- a/tcwg_bmk-build.sh +++ b/tcwg_bmk-build.sh @@ -371,12 +371,13 @@ compare_results () set -euf -o pipefail local metric_id="$1" - local cmp_options="$2" - case "${rr[target]}" in - "arm_eabi") - cmp_options="$cmp_options --has_perf_logs no" - ;; + local compare_opts="" + case "${rr[target]}:$cflags" in + "arm_eabi":*"_LTO"*) compare_opts="--num_symbols 0 --entry_threshold 10 --has_perf_logs no" ;; + *:*"_LTO"*) compare_opts="--num_symbols 0 --entry_threshold 10" ;; + "arm_eabi":*) compare_opts="--has_perf_logs no" ;; + *) compare_opts="" ;; esac if [ -f /usr/lib/linux-tools/install-armhf-perf-workaround.sh ]; then @@ -426,7 +427,7 @@ compare_results () --results_ref $results_ref ++results $results_new \ --top_artifacts "${rr[top_artifacts]}/results-vs-prev" \ --verbose $verbose --hw_tag "$(tcwg_bmk_hw)" \ - $cmp_options \ + $compare_opts \ > ${rr[top_artifacts]}/results-vs-prev/tcwg-benchmark-results.log 2>&1 & local res @@ -520,17 +521,11 @@ no_regression_p () # as failure marker. assert ! [ -f $run_step_artifacts/results.regressions ] - local compare_opts="" - case "$cflags" in - *"_LTO"*) compare_opts="--num_symbols 0 --entry_threshold 10" ;; - esac - # At this stage, # - new/results_id should exist (score>0) # - ref/results_id might not exist, (ex: baseline score<0) # we call compare_results to generate the csv, and check metric regressions. - compare_results "$metric_id" \ - "$compare_opts" + compare_results "$metric_id" if [ -f $run_step_artifacts/results.regressions ]; then return 1 |