From: Paul Eggert <eggert@cs.ucla.edu>
Date: Thu, 29 May 2025 15:30:13 +0000 (-0700)
Subject: cksum: port to Apple LLVM version 10.0.0
X-Git-Tag: v9.8~307
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=4eb2ff1c8984403e7f7def1cf22eab0b0e3c3dad;p=thirdparty%2Fcoreutils.git

cksum: port to Apple LLVM version 10.0.0

This compiler lacks support for the x86-64 VPCLMULQDQ instruction;
compiles fail with "error: invalid cpu feature string for builtin".
Problem reported by Peter Dyballa, and fix suggested by
Pádraig Brady <https://bugs.gnu.org/78562#8>.
* configure.ac (USE_AVX512_CRC32):
Also check that __builtin_cpu_supports ("vpclmulqdq") compiles.
---

diff --git a/NEWS b/NEWS
index 923aa72f88..1f498269f8 100644
--- a/NEWS
+++ b/NEWS
@@ -4,6 +4,10 @@ GNU coreutils NEWS                                    -*- outline -*-
 
 ** Bug fixes
 
+  cksum was not compilable by Apple LLVM 10.0.0 x86-64, which
+  lacks support for checking for the VPCLMULQDQ instruction.
+  [bug introduced in coreutils-9.6]
+
   'cp -p' had spurious "Operation not supported" failures when
   copying to non-NFS files from NFSv4 files with trivial ACLs.
   [bug introduced in coreutils-9.6]
diff --git a/configure.ac b/configure.ac
index 5ea280f262..b446872043 100644
--- a/configure.ac
+++ b/configure.ac
@@ -693,8 +693,9 @@ AC_LINK_IFELSE(
                            0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0);
       a = _mm512_clmulepi64_epi128 (a, b, 0x00);
       a = _mm512_shuffle_epi8 (a, b);
-      return __builtin_cpu_supports ("avx512bw") &&
-        __builtin_cpu_supports ("avx512f");
+      return (__builtin_cpu_supports ("avx512bw")
+              && __builtin_cpu_supports ("avx512f")
+              && __builtin_cpu_supports ("vpclmulqdq"));
     }
   ]])
   ],[