add missing \, standardize cmd arg opts

Author: ianayl

devops/actions/run-tests/benchmark_v2/action.yml

@@ -112,13 +112,13 @@ runs:
         --output-html remote \
         --results-dir "./llvm-ci-perf-results/$RUNNER_NAME" \
         --output-dir "./llvm-ci-perf-results/$RUNNER_NAME" \
-        --preset Minimal
+        --preset Minimal \
         --timestamp-override "$SAVE_TIMESTAMP"
       echo "-----"
       python3 ./devops/scripts/benchmarks/compare.py to_hist \
         --name Baseline_PVC_L0 \
-        --compare_file "./llvm-ci-perf-results/$RUNNER_NAME/results/$SAVE_NAME_$SAVE_TIMESTAMP.json"
-        --results_dir "./llvm-ci-perf-results/$RUNNER_NAME/results/"
+        --compare-file "./llvm-ci-perf-results/$RUNNER_NAME/results/$SAVE_NAME_$SAVE_TIMESTAMP.json"
+        --results-dir "./llvm-ci-perf-results/$RUNNER_NAME/results/"
 
   - name: Push compute-benchmarks results
     if: inputs.upload_results == 'true' && always()

devops/scripts/benchmarks/compare.py

@@ -231,7 +231,7 @@ def to_hist(
     subparsers = parser.add_subparsers(dest="operation", required=True)
     parser_avg = subparsers.add_parser("to_hist", help="Compare a benchmark result to historic average")
     parser_avg.add_argument(
-        "--avg_type",
+        "--avg-type",
         type=str,
         help="Measure of central tendency to use when computing historic average",
         default="median"
@@ -243,13 +243,13 @@ def to_hist(
         help="Save name of the benchmark results to compare to"
     )
     parser_avg.add_argument(
-        "--compare_file",
+        "--compare-file",
         type=str,
         required=True,
         help="Result file to compare against te historic average"
     )
     parser_avg.add_argument(
-        "--results_dir",
+        "--results-dir",
         type=str,
         required=True,
         help="Directory storing results"