Prevent 'functions have mismatched data' warnings

NNPDF · Sep 3, 2024 · f39219f · f39219f
1 parent 3144988
commit f39219f
Show file tree

Hide file tree

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

diff --git a/.github/workflows/python.yml b/.github/workflows/python.yml
@@ -25,7 +25,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)

diff --git a/maintainer/generate-coverage.sh b/maintainer/generate-coverage.sh
@@ -37,7 +37,8 @@ export CARGO_TARGET_DIR="$(mktemp -d)"
 # relative paths sometimes don't work, so use an absolute path
 dir="${CARGO_TARGET_DIR}"/debug/doctestbins
 
-export RUSTFLAGS="-Cinstrument-coverage"
+# `-C link-dead-code` is needed to prevent 'warning: XX functions have mismatched data' warnings
+export RUSTFLAGS="-Cinstrument-coverage -Clink-dead-code"
 export RUSTDOCFLAGS="-Cinstrument-coverage -Z unstable-options --persist-doctests ${dir}"
 
 # -Z doctest-in-workspace is enabled by default starting from 1.72.0