Add -Clink-dead-code to avoid warnings

NNPDF · Sep 5, 2024 · 361dc82 · 361dc82
1 parent dd6c4be
commit 361dc82
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/.github/workflows/capi.yaml b/.github/workflows/capi.yaml
@@ -16,7 +16,8 @@ jobs:
 
       - name: Install PineAPPL's C API
         env:
-          RUSTFLAGS: '-Cinstrument-coverage'
+          # `-C link-dead-code` is needed to prevent 'warning: XX functions have mismatched data' warnings
+          RUSTFLAGS: '-Cinstrument-coverage -Clink-dead-code'
         run: |
           cargo cinstall --verbose --prefix=/usr/local/ --libdir=lib --manifest-path pineappl_capi/Cargo.toml
           ldconfig

diff --git a/.github/workflows/python.yml b/.github/workflows/python.yml
@@ -24,7 +24,8 @@ jobs:
           rustup component add llvm-tools
       - name: Test
         env:
-          RUSTFLAGS: '-Cinstrument-coverage'
+          # `-C link-dead-code` is needed to prevent 'warning: XX functions have mismatched data' warnings
+          RUSTFLAGS: '-Cinstrument-coverage -Clink-dead-code'
         run: |
           cd pineappl_py
           python -m venv env

diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -66,7 +66,8 @@ jobs:
 
       - name: Run tests
         env:
-          RUSTFLAGS: '-Cinstrument-coverage'
+          # `-C link-dead-code` is needed to prevent 'warning: XX functions have mismatched data' warnings
+          RUSTFLAGS: '-Cinstrument-coverage -Clink-dead-code'
         run: |
           # we need stderr, but we can't run test twice because it'll regenerate/modify the binaries which interferes with `llvm-cov`
           cargo test --features=applgrid,evolve,fastnlo,fktable --no-fail-fast 2> >(tee stderr 1>&2)