Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI actions and tests with cargo careful's sanitizer #84

Merged
merged 3 commits into from
Mar 9, 2024
Merged

Update CI actions and tests with cargo careful's sanitizer #84

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ba6f3a6
Remove obsolete lint directive
bluss Mar 9, 2024
1333867
ci: Test with cargo-careful and ThreadSanitizer
bluss Mar 9, 2024
a5c2127
ci: Update github action versions
bluss Mar 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 26 additions & 16 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,14 +56,13 @@ jobs:
features: threading cgemm
test_examples: yes_examples

name: tests/${{ matrix.target }}/${{ matrix.rust }}
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
profile: minimal
toolchain: ${{ matrix.rust }}
target: ${{ matrix.target }}
override: true
targets: ${{ matrix.target }}
- name: Install dependencies
if: matrix.install_deps
run: ${{ matrix.install_deps }}
Expand Down Expand Up @@ -104,14 +103,13 @@ jobs:
experimental: false
target: thumbv6m-none-eabi

name: nostd-build/${{ matrix.target }}/${{ matrix.rust }}
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
profile: minimal
toolchain: ${{ matrix.rust }}
target: ${{ matrix.target }}
override: true
targets: ${{ matrix.target }}
- name: Tests
run: |
cargo rustc "--target=${{ matrix.target }}" --manifest-path=ensure_no_std/Cargo.toml
Expand All @@ -128,14 +126,13 @@ jobs:
target: aarch64-unknown-linux-gnu
features: cgemm

name: cross_test/${{ matrix.target }}/${{ matrix.rust }}
steps:
- uses: actions/checkout@v2
- uses: actions-rs/toolchain@v1
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
profile: minimal
toolchain: ${{ matrix.rust }}
target: ${{ matrix.target }}
override: true
targets: ${{ matrix.target }}
- name: Cache cargo plugins
id: cache
uses: actions/cache@v1
Expand All @@ -151,10 +148,23 @@ jobs:
MMTEST_FAST_TEST: 1
RUSTFLAGS: -Copt-level=2

cargo-careful:
runs-on: ubuntu-latest
name: cargo-careful
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
toolchain: nightly
- uses: Swatinem/rust-cache@v2
- name: Install cargo-careful
run: cargo install cargo-careful
- run: cargo careful test -Zcareful-sanitizer=thread --features=threading,cgemm

miri:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- name: Miri
run: ci/miri.sh --features cgemm

2 changes: 0 additions & 2 deletions src/ptr.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![allow(unknown_lints, suspicious_auto_trait_impls)]

use rawpointer::PointerExt;

/// A Send + Sync raw pointer wrapper
Expand Down
Loading