git: 17c13889a513 - main - textproc/krep: Disable all SIMD hacks/detections in favor of CPUTYPE

From: Älven <alven_at_FreeBSD.org>
Date: Sat, 21 Feb 2026 06:13:55 UTC
The branch main has been updated by alven:

URL: https://cgit.FreeBSD.org/ports/commit/?id=17c13889a5137e45011a083759a1879fec109613

commit 17c13889a5137e45011a083759a1879fec109613
Author:     Älven <alven@FreeBSD.org>
AuthorDate: 2026-02-21 05:53:05 +0000
Commit:     Älven <alven@FreeBSD.org>
CommitDate: 2026-02-21 06:13:38 +0000

    textproc/krep: Disable all SIMD hacks/detections in favor of CPUTYPE
    
    All SIMD "hacks/detection" should be disabled as we have CPUTYPE and
    the current approach is also broken as host might not share the same
    CPU features as target.
    
    Reported by:    diizzy@ (via e-mail)
    Approved by:    db@, yuri@ (Mentors, implicit)
---
 textproc/krep/Makefile             |  5 -----
 textproc/krep/files/patch-Makefile | 22 +++++++++-------------
 2 files changed, 9 insertions(+), 18 deletions(-)

diff --git a/textproc/krep/Makefile b/textproc/krep/Makefile
index 9b348d3c46d8..951dbdc69c80 100644
--- a/textproc/krep/Makefile
+++ b/textproc/krep/Makefile
@@ -11,7 +11,6 @@ LICENSE=	BSD2CLAUSE
 LICENSE_FILE=	${WRKSRC}/LICENSE
 
 USES=		gmake
-
 USE_GITHUB=	yes
 GH_ACCOUNT=	davidesantangelo
 
@@ -19,8 +18,4 @@ TEST_TARGET=	test
 
 PLIST_FILES=	bin/krep
 
-OPTIONS_DEFINE=	SIMD
-
-SIMD_MAKE_ARGS=	SIMD=1
-
 .include <bsd.port.mk>
diff --git a/textproc/krep/files/patch-Makefile b/textproc/krep/files/patch-Makefile
index fc763f843ff5..32f8eaefbddd 100644
--- a/textproc/krep/files/patch-Makefile
+++ b/textproc/krep/files/patch-Makefile
@@ -1,6 +1,6 @@
---- Makefile.orig	2026-02-18 16:14:00 UTC
+--- Makefile.orig	2026-02-18 15:31:31 UTC
 +++ Makefile
-@@ -5,10 +5,10 @@ BINDIR = $(PREFIX)/bin
+@@ -5,11 +5,15 @@ BINDIR = $(PREFIX)/bin
  PREFIX ?= /usr/local
  BINDIR = $(PREFIX)/bin
  
@@ -12,23 +12,19 @@
 -LDFLAGS = -pthread -flto
 +LDFLAGS += -pthread -flto
  
++# All SIMD "hacks/detection" should be disabled as we have CPUTYPE and the current
++# approach is also broken as host might not share the same CPU features as target.
++ifeq (0, 1)
++
  # Build mode: set NATIVE=1 for maximum performance on local machine
  # Example: make NATIVE=1
-@@ -19,6 +19,9 @@ ARCH := $(shell uname -m)
- # Detect architecture for SIMD flags
- ARCH := $(shell uname -m)
- 
-+# Build mode: set SIMD=1 for maximum performance using SIMD instructions
-+# Example: make SIMD=1
-+ifdef SIMD
- ifeq ($(ARCH), x86_64)
-     # Check for AVX-512 support (Linux: /proc/cpuinfo, macOS: sysctl)
-     HAS_AVX512 := $(shell (grep -q avx512f /proc/cpuinfo 2>/dev/null && echo 1) || \
-@@ -46,6 +49,7 @@ else ifeq ($(ARCH), aarch64)
+ ifdef NATIVE
+@@ -46,6 +50,8 @@ else ifeq ($(ARCH), aarch64)
  else ifeq ($(ARCH), aarch64)
      # Enable NEON for aarch64 Linux
      CFLAGS += -D__ARM_NEON
 +endif
++
  endif
  
  # Source files