Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 23 Apr 2024 22:14:35 GMT
From:      Brooks Davis <brooks@FreeBSD.org>
To:        ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-branches@FreeBSD.org
Subject:   git: 3645478a263b - 2024Q2 - devel/llvm*: Remove unnecessary C/CFLAGS definitions
Message-ID:  <202404232214.43NMEZEZ031983@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch 2024Q2 has been updated by brooks:

URL: https://cgit.FreeBSD.org/ports/commit/?id=3645478a263b3af06d158a2fadebab0490c0abb0

commit 3645478a263b3af06d158a2fadebab0490c0abb0
Author:     Daniel Engberg <diizzy@FreeBSD.org>
AuthorDate: 2024-04-14 19:32:42 +0000
Commit:     Brooks Davis <brooks@FreeBSD.org>
CommitDate: 2024-04-23 22:14:14 +0000

    devel/llvm*: Remove unnecessary C/CFLAGS definitions
    
    As of 3c5189a0fd4c82addfb659024bf28c5006bf88f7
    there's no need to define DNDEBUG manually
    
    PR:             277909
    Approved by:    portmgr (maintainer timeout, 2+ weeks)
    
    (cherry picked from commit 1f50c8e24264be1d64a96e84536e2e1d59b63180)
---
 devel/llvm11/Makefile | 5 -----
 devel/llvm12/Makefile | 5 -----
 devel/llvm13/Makefile | 5 -----
 devel/llvm14/Makefile | 5 -----
 devel/llvm15/Makefile | 5 -----
 devel/llvm16/Makefile | 5 -----
 devel/llvm17/Makefile | 5 -----
 devel/llvm18/Makefile | 5 -----
 8 files changed, 40 deletions(-)

diff --git a/devel/llvm11/Makefile b/devel/llvm11/Makefile
index 90debe37315c..5a2587e4d02f 100644
--- a/devel/llvm11/Makefile
+++ b/devel/llvm11/Makefile
@@ -54,11 +54,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_Z3_SOLVER=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG DOCS EXTRAS LIT LLD LLDB PYCLANG
 OPTIONS_DEFINE_aarch64=	OPENMP
 OPTIONS_DEFINE_amd64=	COMPILER_RT GOLD OPENMP
diff --git a/devel/llvm12/Makefile b/devel/llvm12/Makefile
index 1930a23dc473..73d1f2e86b2d 100644
--- a/devel/llvm12/Makefile
+++ b/devel/llvm12/Makefile
@@ -64,11 +64,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG DOCS EXTRAS LIT LLD PYCLANG
 OPTIONS_DEFINE_aarch64=	OPENMP
 OPTIONS_DEFINE_amd64=	COMPILER_RT GOLD LLDB OPENMP
diff --git a/devel/llvm13/Makefile b/devel/llvm13/Makefile
index 955b78d82c73..414a74af375d 100644
--- a/devel/llvm13/Makefile
+++ b/devel/llvm13/Makefile
@@ -73,11 +73,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG DOCS EXTRAS FLANG LIT LLD LLDB MLIR PYCLANG
 OPTIONS_DEFINE_aarch64=	OPENMP
 OPTIONS_DEFINE_amd64=	COMPILER_RT GOLD OPENMP
diff --git a/devel/llvm14/Makefile b/devel/llvm14/Makefile
index ec7749aa3d93..25c95a6fb6b2 100644
--- a/devel/llvm14/Makefile
+++ b/devel/llvm14/Makefile
@@ -77,11 +77,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG DOCS EXTRAS FLANG LIT LLD LLDB MLIR PYCLANG
 OPTIONS_DEFINE_aarch64=	OPENMP
 OPTIONS_DEFINE_amd64=	COMPILER_RT GOLD OPENMP
diff --git a/devel/llvm15/Makefile b/devel/llvm15/Makefile
index 91ef79f08208..900606601e2a 100644
--- a/devel/llvm15/Makefile
+++ b/devel/llvm15/Makefile
@@ -84,11 +84,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG DOCS LLD STATIC_LIBS
 OPTIONS_DEFAULT=	BE_AMDGPU BE_WASM CLANG LLD
 OPTIONS_SINGLE=		BACKENDS
diff --git a/devel/llvm16/Makefile b/devel/llvm16/Makefile
index c9ee00d8f6fa..d8b29aa36f97 100644
--- a/devel/llvm16/Makefile
+++ b/devel/llvm16/Makefile
@@ -85,11 +85,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG COMPILER_RT DOCS LLD STATIC_LIBS
 OPTIONS_DEFAULT=	BE_AMDGPU BE_WASM CLANG LLD
 OPTIONS_SINGLE=		BACKENDS
diff --git a/devel/llvm17/Makefile b/devel/llvm17/Makefile
index fd4faad41a89..2a0873547258 100644
--- a/devel/llvm17/Makefile
+++ b/devel/llvm17/Makefile
@@ -88,11 +88,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 # LLVM likes to pick up ${LOCALBASE}/lib/libtinfo.so.*
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG COMPILER_RT DOCS LLD STATIC_LIBS
 OPTIONS_DEFAULT=	BE_AMDGPU BE_WASM CLANG LLD
 OPTIONS_SINGLE=		BACKENDS
diff --git a/devel/llvm18/Makefile b/devel/llvm18/Makefile
index 7b831a79d372..f28a46370c65 100644
--- a/devel/llvm18/Makefile
+++ b/devel/llvm18/Makefile
@@ -86,11 +86,6 @@ CMAKE_ARGS+=	-DLLVM_ENABLE_LIBEDIT=OFF
 CMAKE_ARGS+=	-DLLVM_ENABLE_TERMINFO=OFF
 CMAKE_ARGS+=	-DLLVM_VERSION_SUFFIX=
 
-# Disable assertions.  They should be disabled by cmake, but USES=cmake
-# overrides -DCMAKE_*_FLAGS_RELEASE.
-CFLAGS+=	-DNDEBUG
-CXXFLAGS+=	-DNDEBUG
-
 OPTIONS_DEFINE=	BE_AMDGPU BE_WASM CLANG COMPILER_RT DOCS LLD STATIC_LIBS
 OPTIONS_DEFAULT=	BE_AMDGPU BE_WASM CLANG LLD
 OPTIONS_SINGLE=		BACKENDS



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202404232214.43NMEZEZ031983>