try use matrix to optimize sanitize speed

PureWhiteWu · Feb 5, 2025 · fef0b7b · fef0b7b
1 parent 2e59d9a
commit fef0b7b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -53,6 +53,12 @@ jobs:
           cargo fmt -- --check
 
   sanitize:
+    strategy:
+      fail-fast: false
+      matrix:
+        san: [address, leak]
+        feature: ["", "arbitrary_precision", "sort_keys", "use_raw", "utf8_lossy"]
+    name: Sanitize ${{matrix.san}} feature ${{matrix.feature}}
     runs-on: [self-hosted, Linux, amd64]
     steps:
     - uses: actions/checkout@v4

diff --git a/scripts/sanitize.sh b/scripts/sanitize.sh
@@ -11,9 +11,5 @@ run_tests() {
     cargo +nightly test --doc --package sonic-rs --target x86_64-unknown-linux-gnu --features "$features" -- --show-output --test-threads=1
 }
 
-for san in address leak; do
-    for feature in "" "arbitrary_precision" "sort_keys" "use_raw" "utf8_lossy"; do
-        echo "Running tests with $san and $feature"
-        RUSTFLAGS="-Zsanitizer=${san}" RUSTDOCFLAGS="-Zsanitizer=${san}" run_tests $san $feature
-    done
-done
+echo "Running tests with $san and $feature"
+RUSTFLAGS="-Zsanitizer=$1" RUSTDOCFLAGS="-Zsanitizer=$2" run_tests $1 $2