From c98ba8dcfcce77d78cd68dedfa8c2bfd17ff01fc Mon Sep 17 00:00:00 2001 From: Maxim Kuvyrkov Date: Thu, 21 Mar 2019 09:55:00 +0000 Subject: tcwg_bmk-build.sh: Consider only final executable size for regressions Switch from considering top 10 symbols for regressions to only size of final executable. We will start tracking top 1-2 symbols after stabilizing tracking of final executables. This also allows us to switch to quick "verify" benchmarking mode. Change-Id: If8f1dd14c9ab861e1b0c1d617790df71d4ba25d1 --- tcwg-benchmark-results.sh | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'tcwg-benchmark-results.sh') diff --git a/tcwg-benchmark-results.sh b/tcwg-benchmark-results.sh index 3f3e2617..8f0bf337 100755 --- a/tcwg-benchmark-results.sh +++ b/tcwg-benchmark-results.sh @@ -2,7 +2,7 @@ # Clean: shellcheck -e 2001 ./tcwg-benchmark-results.sh -set -ef -o pipefail +set -euf -o pipefail scripts=$(dirname $0) . $scripts/jenkins-helpers.sh @@ -13,10 +13,7 @@ obligatory_variables results_id results_ref top_artifacts="${top_artifacts-artifacts}" verbose="${verbose-true}" - -# Jenkins doesn't define variables when parameter value is empty (like cflags), -# so enable "set -u" only after above binding of variables. -set -u +num_entries_opt="${num_dsos+--num-dsos $num_dsos} ${num_symbols+--num-symbols $num_symbols}" if $verbose; then set -x; fi @@ -26,7 +23,10 @@ num=0 csvs="" for i in $results_ref $results_id; do rsync -az --delete "$results_top-$i/" results-$num/ - $scripts/../bmk-scripts/perfdatadir2csv.sh --buildid-dir local --format sample,size --sort-field sample --results-dir "results-$num/" > "results-$num.csv" + $scripts/../bmk-scripts/perfdatadir2csv.sh \ + --buildid-dir local --format sample,size --sort-field sample \ + $num_entries_opt \ + --results-dir "results-$num/" > "results-$num.csv" csvs="$csvs results-$num.csv" num=$(($num+1)) done -- cgit v1.2.3