Only run Valgrind on ci jobs we choose ci (LLVM 20) (#760)

* Only run tests under Valgrind for specific ci jobs

* Fix conditional check for Valgrind in action.yml

* Fix syntax for accessing matrix variable in action.yml

Author: mcbarton

.github/actions/Build_and_Test_CppInterOp/action.yml

@@ -58,7 +58,7 @@ runs:
         else
           cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }}
           os="${{ matrix.os }}"
-          if [[ "${os}" != "macos"* ]]; then
+          if [[ "${os}" != "macos"* && "${{ matrix.Valgrind }}" == "On" ]]; then
             CLANG_VERSION="${{ matrix.clang-runtime }}"
             if [[ "$CLANG_VERSION" == "20" && "${{ matrix.cling }}" == "Off" ]]; then
               SUPPRESSION_FILE="../etc/clang${CLANG_VERSION}-valgrind.supp"

.github/actions/Build_and_Test_cppyy/action.yml

@@ -106,7 +106,7 @@ runs:
 
         set -o pipefail
         os="${{ matrix.os }}"
-        if [[ "${os}" != "macos"* ]]; then
+        if [[ "${os}" != "macos"* && "${{ matrix.Valgrind }}" == "On" ]]; then
           echo "Running valgrind on passing tests"
           CLANG_VERSION="${{ matrix.clang-runtime }}"
 

.github/workflows/main.yml

@@ -31,6 +31,7 @@ jobs:
             cppyy: Off
             llvm_enable_projects: "clang"
             llvm_targets_to_build: "host;NVPTX"
+            Valgrind: On
           - name: ubu24-arm-gcc12-clang-repl-19-cppyy
             os: ubuntu-24.04-arm
             compiler: gcc-12
@@ -67,6 +68,7 @@ jobs:
             llvm_targets_to_build: "host;NVPTX"
             coverage: true
             oop-jit: On
+            Valgrind: On
           - name: ubu24-x86-gcc12-clang-repl-20
             os: ubuntu-24.04
             compiler: gcc-12
@@ -311,3 +313,4 @@ jobs:
       uses: mxschmitt/action-tmate@v3
       # When debugging increase to a suitable value!
       timeout-minutes: 30
+