Browse Source

CI: fix rebuilding peepmatic peephole optimizers (#2311)

The test that triggers the rebuild of the peephole optimizers is in the
`cranelift-codegen` crate, not the umbrella cranelift crate. This was previously
successfully running zero tests, and then successfully reporting no `git diff`
because no peephole optimizers were ever rebuilt.

This change fixes it so that we run the correct test that triggers the
rebuilding of the peephole optimizers.
pull/2314/head
Nick Fitzgerald 4 years ago
committed by GitHub
parent
commit
4f104d3a4e
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      .github/workflows/main.yml

4
.github/workflows/main.yml

@ -168,9 +168,9 @@ jobs:
- name: Rebuild Peepmatic-based peephole optimizers and test them - name: Rebuild Peepmatic-based peephole optimizers and test them
run: | run: |
cargo test \ cargo test \
--features 'enable-peepmatic cranelift-codegen/rebuild-peephole-optimizers' \ --features 'enable-peepmatic rebuild-peephole-optimizers' \
peepmatic peepmatic
working-directory: ./cranelift working-directory: ./cranelift/codegen
- name: Check that built peephole optimizers are up to date - name: Check that built peephole optimizers are up to date
run: git diff --exit-code run: git diff --exit-code
- name: Test with Peepmatic-based peephole optimizers - name: Test with Peepmatic-based peephole optimizers

Loading…
Cancel
Save