fix: icc < 22.0.1 advertises but doesn't support the fallthrough attr…

…ibute
nemequ · Sep 16, 2024 · d5f482f · d5f482f
1 parent 7c44b9e
commit d5f482f
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 16 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -56,20 +56,20 @@ jobs:
       fail-fast: false
       matrix:
         version:
-        - "2021.1.1"
-        - "2021.1.2"
-        - "2021.2.0"
-        - "2021.3.0"
+        # - "2021.1.1"
+        # - "2021.1.2"
+        # - "2021.2.0"
+        # - "2021.3.0"
         - "2021.4.0"
         - "2022.0.1"
-        - "2022.0.2"
-        - "2022.1.0"
-        - "2022.2.0"
-        - "2022.2.1"
-        - "2023.0.0"
-        - "2023.1.0"
-        - "2023.2.0"
-        - "2023.2.1"
+        # - "2022.0.2"
+        # - "2022.1.0"
+        # - "2022.2.0"
+        # - "2022.2.1"
+        # - "2023.0.0"
+        # - "2023.1.0"
+        # - "2023.2.0"
+        # - "2023.2.1"
     steps:
       - uses: actions/checkout@v4
       - name: Free some disk space
@@ -86,16 +86,16 @@ jobs:
           . /opt/intel/oneapi/setvars.sh
           icc --version
           icc -dM -E - </dev/null
-      # - name: tmate session
-      #   uses: mxschmitt/action-tmate@v3
+      - name: tmate session
+        uses: mxschmitt/action-tmate@v3
       - name: Compile
         env:
           CC: icc
           CXX: icpc
         working-directory: test
         run: |
           . /opt/intel/oneapi/setvars.sh
-          make likely
+          make
 
   # nvhpc:
   #   runs-on: ubuntu-latest

diff --git a/hedley.h b/hedley.h
@@ -1727,7 +1727,10 @@ HEDLEY_DIAGNOSTIC_POP
 # undef HEDLEY_FALL_THROUGH
 #endif
 #if \
-  HEDLEY_HAS_ATTRIBUTE(fallthrough) || \
+  ( \
+    HEDLEY_HAS_ATTRIBUTE(fallthrough) && \
+    (!defined(HEDLEY_INTEL_VERSION) || HEDLEY_INTEL_VERSION_CHECK(20,22,0)) \
+  ) || \
   HEDLEY_GCC_VERSION_CHECK(7,0,0) || \
   HEDLEY_MCST_LCC_VERSION_CHECK(1,25,10)
 #  define HEDLEY_FALL_THROUGH __attribute__((__fallthrough__))