Skip to content

Commit 6366d2e

Browse files
Run bench integrations tests only on PR - in a sep. step
1 parent 4b27d65 commit 6366d2e

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

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

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,8 @@ runs:
255255
# Build and run benchmarks
256256
257257
echo "::group::setup_workdir"
258-
WORKDIR="$(realpath ./llvm_test_workdir)"
258+
export WORKDIR="$(realpath ./llvm_test_workdir)"
259+
echo "WORKDIR=$WORKDIR" >> $GITHUB_ENV
259260
if [ -n "$WORKDIR" ] && [ -d "$WORKDIR" ] && [[ "$WORKDIR" == *llvm_test_workdir* ]]; then rm -rf "$WORKDIR" ; fi
260261
261262
# Clean up potentially existing, old summary files
@@ -297,19 +298,21 @@ runs:
297298
${{ inputs.dry_run == 'true' && '--dry-run' || '' }} \
298299
299300
echo "::endgroup::"
300-
301+
- name: Run benchmarks integration tests
302+
shell: bash
303+
if: ${{ github.event_name == 'pull_request' }}
304+
run: |
301305
# Run benchmarks' integration tests
306+
302307
# NOTE: Each integration test prints its own group name as part of test script
303-
if [ '${{ github.event_name == 'pull_request' }}' = 'true' ]; then
304-
export LLVM_BENCHMARKS_UNIT_TESTING=1
305-
export COMPUTE_BENCHMARKS_BUILD_PATH=$WORKDIR/compute-benchmarks-build
306-
python3 ./devops/scripts/benchmarks/tests/test_integration.py
307-
fi
308+
export LLVM_BENCHMARKS_UNIT_TESTING=1
309+
export COMPUTE_BENCHMARKS_BUILD_PATH=$WORKDIR/compute-benchmarks-build
310+
python3 ./devops/scripts/benchmarks/tests/test_integration.py
308311
- name: Cache changes and upload github summary
309312
if: always()
310313
shell: bash
311314
run: |
312-
# Cache changes and upload github summaries
315+
# Upload github summaries and cache changes
313316
[ -f "github_summary_exe.md" ] && cat github_summary_exe.md >> $GITHUB_STEP_SUMMARY
314317
[ -f "github_summary_reg.md" ] && cat github_summary_reg.md >> $GITHUB_STEP_SUMMARY
315318

0 commit comments

Comments
 (0)