Skip to content

Commit 349a611

Browse files
committed
Delete travis config, move tests to github actions.
1 parent 5b4282e commit 349a611

File tree

4 files changed

+39
-86
lines changed

4 files changed

+39
-86
lines changed

.github/workflows/ci.yml

+33-2
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,7 @@ jobs:
9494
- { target: i586-pc-windows-msvc, target_feature: +sse2, os: windows-latest }
9595

9696
# Annoyingly, the x86_64-unknown-linux-gnu runner *almost* always has
97-
# avx512vl, but occasionally doesn't. As a result, we still run that
98-
# one under travis.
97+
# avx512vl, but occasionally doesn't. Maybe one day we can enable it.
9998

10099
steps:
101100
- uses: actions/checkout@v2
@@ -141,6 +140,31 @@ jobs:
141140
- name: Test (release)
142141
run: cargo test --verbose --target=${{ matrix.target }} --release
143142

143+
wasm-tests:
144+
name: "wasm (firefox, ${{ matrix.name }})"
145+
runs-on: ubuntu-latest
146+
strategy:
147+
matrix:
148+
include:
149+
- { name: default, RUSTFLAGS: "" }
150+
- { name: simd128, RUSTFLAGS: "-C target-feature=+simd128" }
151+
steps:
152+
- uses: actions/checkout@v2
153+
- name: Setup Rust
154+
run: |
155+
rustup update nightly --no-self-update
156+
rustup default nightly
157+
- name: Install wasm-pack
158+
run: curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh
159+
- name: Test (debug)
160+
run: wasm-pack test --firefox --headless crates/core_simd
161+
env:
162+
RUSTFLAGS: ${{ matrix.rustflags }}
163+
- name: Test (release)
164+
run: wasm-pack test --firefox --headless crates/core_simd --release
165+
env:
166+
RUSTFLAGS: ${{ matrix.rustflags }}
167+
144168
cross-tests:
145169
name: "${{ matrix.target }} (via cross)"
146170
runs-on: ubuntu-latest
@@ -163,12 +187,19 @@ jobs:
163187
# 32-bit arm has a few idiosyncracies like having subnormal flushing
164188
# to zero on by default. Ideally we'd set
165189
- armv7-unknown-linux-gnueabihf
190+
- aarch64-unknown-linux-gnu
166191
# Note: The issue above means neither of these mips targets will use
167192
# MSA (mips simd) but MIPS uses a nonstandard binary representation
168193
# for NaNs which makes it worth testing on despite that.
169194
- mips-unknown-linux-gnu
170195
- mips64-unknown-linux-gnuabi64
171196
- riscv64gc-unknown-linux-gnu
197+
# TODO this test works, but it appears to time out
198+
# - powerpc-unknown-linux-gnu
199+
# TODO this test is broken, but it appears to be a problem with QEMU, not us.
200+
# - powerpc64le-unknown-linux-gnu
201+
# TODO enable this once a new version of cross is released
202+
# - powerpc64-unknown-linux-gnu
172203

173204
steps:
174205
- uses: actions/checkout@v2

.travis.yml

-82
This file was deleted.

crates/core_simd/src/masks/full_masks.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ where
119119
// There is a bug where LLVM appears to implement this operation with the wrong
120120
// bit order.
121121
// TODO fix this in a better way
122-
if cfg!(any(target_arch = "mips", target_arch = "mips64")) {
122+
if cfg!(target_endian = "big") {
123123
for x in bitmask.as_mut() {
124124
*x = x.reverse_bits();
125125
}
@@ -136,7 +136,7 @@ where
136136
// There is a bug where LLVM appears to implement this operation with the wrong
137137
// bit order.
138138
// TODO fix this in a better way
139-
if cfg!(any(target_arch = "mips", target_arch = "mips64")) {
139+
if cfg!(target_endian = "big") {
140140
for x in bitmask.as_mut() {
141141
*x = x.reverse_bits();
142142
}

crates/core_simd/tests/masks.rs

+4
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,10 @@ macro_rules! test_mask_api {
8686

8787
mod mask_api {
8888
test_mask_api! { i8 }
89+
test_mask_api! { i16 }
90+
test_mask_api! { i32 }
91+
test_mask_api! { i64 }
92+
test_mask_api! { isize }
8993
}
9094

9195
#[test]

0 commit comments

Comments
 (0)