[libc-commits] [libc] [libc] Remove optimization flags on entrypoints (PR #165782)
via libc-commits
libc-commits at lists.llvm.org
Thu Oct 30 14:30:54 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-libc
Author: Michael Jones (michaelrj-google)
<details>
<summary>Changes</summary>
Optimization flags are now handled through a common flag. These are no
longer necessary.
Fixes #<!-- -->112409
---
Full diff: https://github.com/llvm/llvm-project/pull/165782.diff
4 Files Affected:
- (modified) libc/src/fenv/CMakeLists.txt (-32)
- (modified) libc/src/math/amdgpu/CMakeLists.txt (-70)
- (modified) libc/src/math/generic/CMakeLists.txt (+1-27)
- (modified) libc/src/math/nvptx/CMakeLists.txt (-58)
``````````diff
diff --git a/libc/src/fenv/CMakeLists.txt b/libc/src/fenv/CMakeLists.txt
index c5431b1b9d55e..f368845977964 100644
--- a/libc/src/fenv/CMakeLists.txt
+++ b/libc/src/fenv/CMakeLists.txt
@@ -6,8 +6,6 @@ add_entrypoint_object(
fegetround.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -18,8 +16,6 @@ add_entrypoint_object(
fesetround.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -30,8 +26,6 @@ add_entrypoint_object(
feclearexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -42,8 +36,6 @@ add_entrypoint_object(
feraiseexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -54,8 +46,6 @@ add_entrypoint_object(
fetestexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -67,8 +57,6 @@ add_entrypoint_object(
DEPENDS
libc.hdr.types.fexcept_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -80,8 +68,6 @@ add_entrypoint_object(
DEPENDS
libc.hdr.types.fenv_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -93,8 +79,6 @@ add_entrypoint_object(
DEPENDS
libc.hdr.types.fenv_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -107,8 +91,6 @@ add_entrypoint_object(
libc.hdr.fenv_macros
libc.hdr.types.fexcept_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -119,8 +101,6 @@ add_entrypoint_object(
fesetexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -133,8 +113,6 @@ add_entrypoint_object(
libc.hdr.fenv_macros
libc.hdr.types.fexcept_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -147,8 +125,6 @@ add_entrypoint_object(
libc.hdr.fenv_macros
libc.hdr.types.fenv_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -161,8 +137,6 @@ add_entrypoint_object(
libc.hdr.fenv_macros
libc.hdr.types.fenv_t
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -173,8 +147,6 @@ add_entrypoint_object(
feenableexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -185,8 +157,6 @@ add_entrypoint_object(
fedisableexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -197,6 +167,4 @@ add_entrypoint_object(
fegetexcept.h
DEPENDS
libc.src.__support.FPUtil.fenv_impl
- COMPILE_OPTIONS
- -O2
)
diff --git a/libc/src/math/amdgpu/CMakeLists.txt b/libc/src/math/amdgpu/CMakeLists.txt
index e2cd3b99c3037..d05d519b74b4f 100644
--- a/libc/src/math/amdgpu/CMakeLists.txt
+++ b/libc/src/math/amdgpu/CMakeLists.txt
@@ -4,8 +4,6 @@ add_entrypoint_object(
ceil.cpp
HDRS
../ceil.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -14,8 +12,6 @@ add_entrypoint_object(
ceilf.cpp
HDRS
../ceilf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -24,8 +20,6 @@ add_entrypoint_object(
copysign.cpp
HDRS
../copysign.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -34,8 +28,6 @@ add_entrypoint_object(
copysignf.cpp
HDRS
../copysignf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -44,8 +36,6 @@ add_entrypoint_object(
fabs.cpp
HDRS
../fabs.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -54,8 +44,6 @@ add_entrypoint_object(
fabsf.cpp
HDRS
../fabsf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -64,8 +52,6 @@ add_entrypoint_object(
floor.cpp
HDRS
../floor.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -74,8 +60,6 @@ add_entrypoint_object(
floorf.cpp
HDRS
../floorf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -84,8 +68,6 @@ add_entrypoint_object(
fma.cpp
HDRS
../fma.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -94,8 +76,6 @@ add_entrypoint_object(
fmaf.cpp
HDRS
../fmaf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -104,8 +84,6 @@ add_entrypoint_object(
fmax.cpp
HDRS
../fmax.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -114,8 +92,6 @@ add_entrypoint_object(
fmaxf.cpp
HDRS
../fmaxf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -124,8 +100,6 @@ add_entrypoint_object(
fmin.cpp
HDRS
../fmin.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -134,8 +108,6 @@ add_entrypoint_object(
fminf.cpp
HDRS
../fminf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -144,8 +116,6 @@ add_entrypoint_object(
fmod.cpp
HDRS
../fmod.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -154,8 +124,6 @@ add_entrypoint_object(
fmodf.cpp
HDRS
../fmodf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -164,8 +132,6 @@ add_entrypoint_object(
nearbyint.cpp
HDRS
../nearbyint.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -174,8 +140,6 @@ add_entrypoint_object(
nearbyintf.cpp
HDRS
../nearbyintf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -184,8 +148,6 @@ add_entrypoint_object(
remainder.cpp
HDRS
../remainder.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -194,8 +156,6 @@ add_entrypoint_object(
remainderf.cpp
HDRS
../remainderf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -204,8 +164,6 @@ add_entrypoint_object(
rint.cpp
HDRS
../rint.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -214,8 +172,6 @@ add_entrypoint_object(
rintf.cpp
HDRS
../rintf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -224,8 +180,6 @@ add_entrypoint_object(
round.cpp
HDRS
../round.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -234,8 +188,6 @@ add_entrypoint_object(
sqrt.cpp
HDRS
../sqrt.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -244,8 +196,6 @@ add_entrypoint_object(
sqrtf.cpp
HDRS
../sqrtf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -254,8 +204,6 @@ add_entrypoint_object(
trunc.cpp
HDRS
../trunc.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -264,8 +212,6 @@ add_entrypoint_object(
truncf.cpp
HDRS
../truncf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -274,8 +220,6 @@ add_entrypoint_object(
frexp.cpp
HDRS
../frexp.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -284,8 +228,6 @@ add_entrypoint_object(
frexpf.cpp
HDRS
../frexpf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -294,8 +236,6 @@ add_entrypoint_object(
scalbn.cpp
HDRS
../scalbn.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -304,8 +244,6 @@ add_entrypoint_object(
scalbnf.cpp
HDRS
../scalbnf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -314,8 +252,6 @@ add_entrypoint_object(
ldexp.cpp
HDRS
../ldexp.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -324,8 +260,6 @@ add_entrypoint_object(
ldexpf.cpp
HDRS
../ldexpf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -336,7 +270,6 @@ add_entrypoint_object(
../tgamma.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -347,7 +280,6 @@ add_entrypoint_object(
../tgammaf.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -358,7 +290,6 @@ add_entrypoint_object(
../lgamma.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -369,5 +300,4 @@ add_entrypoint_object(
../lgamma_r.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
diff --git a/libc/src/math/generic/CMakeLists.txt b/libc/src/math/generic/CMakeLists.txt
index 6068c36e558ef..c048a64db6bc2 100644
--- a/libc/src/math/generic/CMakeLists.txt
+++ b/libc/src/math/generic/CMakeLists.txt
@@ -2662,8 +2662,6 @@ add_entrypoint_object(
../fmaximum_mag.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2674,8 +2672,6 @@ add_entrypoint_object(
../fmaximum_magf.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2686,8 +2682,6 @@ add_entrypoint_object(
../fmaximum_magl.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2735,8 +2729,6 @@ add_entrypoint_object(
../fmaximum_mag_num.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2747,8 +2739,6 @@ add_entrypoint_object(
../fmaximum_mag_numf.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2759,8 +2749,6 @@ add_entrypoint_object(
../fmaximum_mag_numl.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2954,8 +2942,6 @@ add_entrypoint_object(
../fminimum_mag.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2966,8 +2952,6 @@ add_entrypoint_object(
../fminimum_magf.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -2978,8 +2962,6 @@ add_entrypoint_object(
../fminimum_magl.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -3027,8 +3009,6 @@ add_entrypoint_object(
../fminimum_mag_num.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -3039,8 +3019,6 @@ add_entrypoint_object(
../fminimum_mag_numf.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -3051,8 +3029,6 @@ add_entrypoint_object(
../fminimum_mag_numl.h
DEPENDS
libc.src.__support.FPUtil.basic_operations
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -4306,7 +4282,7 @@ add_entrypoint_object(
libc.hdr.errno_macros
libc.hdr.fenv_macros
libc.src.__support.FPUtil.except_value_utils
- libc.src.__support.FPUtil.fenv_impl
+ libc.src.__support.FPUtil.fenv_impl
libc.src.__support.FPUtil.fp_bits
libc.src.__support.FPUtil.rounding_mode
libc.src.__support.macros.optimization
@@ -4546,8 +4522,6 @@ add_entrypoint_object(
atan.cpp
HDRS
../atan.h
- COMPILE_OPTIONS
- -O3
DEPENDS
libc.src.__support.math.atan
)
diff --git a/libc/src/math/nvptx/CMakeLists.txt b/libc/src/math/nvptx/CMakeLists.txt
index fcb2870b4bb1c..e27c316ff20ca 100644
--- a/libc/src/math/nvptx/CMakeLists.txt
+++ b/libc/src/math/nvptx/CMakeLists.txt
@@ -4,8 +4,6 @@ add_entrypoint_object(
ceil.cpp
HDRS
../ceil.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -14,8 +12,6 @@ add_entrypoint_object(
ceilf.cpp
HDRS
../ceilf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -24,8 +20,6 @@ add_entrypoint_object(
copysign.cpp
HDRS
../copysign.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -34,8 +28,6 @@ add_entrypoint_object(
copysignf.cpp
HDRS
../copysignf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -44,8 +36,6 @@ add_entrypoint_object(
fabs.cpp
HDRS
../fabs.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -54,8 +44,6 @@ add_entrypoint_object(
fabsf.cpp
HDRS
../fabsf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -64,8 +52,6 @@ add_entrypoint_object(
floor.cpp
HDRS
../floor.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -74,8 +60,6 @@ add_entrypoint_object(
floorf.cpp
HDRS
../floorf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -84,8 +68,6 @@ add_entrypoint_object(
fma.cpp
HDRS
../fma.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -94,8 +76,6 @@ add_entrypoint_object(
fmaf.cpp
HDRS
../fmaf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -104,8 +84,6 @@ add_entrypoint_object(
fmax.cpp
HDRS
../fmax.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -114,8 +92,6 @@ add_entrypoint_object(
fmaxf.cpp
HDRS
../fmaxf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -124,8 +100,6 @@ add_entrypoint_object(
fmin.cpp
HDRS
../fmin.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -134,8 +108,6 @@ add_entrypoint_object(
fminf.cpp
HDRS
../fminf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -144,8 +116,6 @@ add_entrypoint_object(
fmod.cpp
HDRS
../fmod.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -154,8 +124,6 @@ add_entrypoint_object(
fmodf.cpp
HDRS
../fmodf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -164,8 +132,6 @@ add_entrypoint_object(
nearbyint.cpp
HDRS
../nearbyint.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -174,8 +140,6 @@ add_entrypoint_object(
nearbyintf.cpp
HDRS
../nearbyintf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -184,8 +148,6 @@ add_entrypoint_object(
remainder.cpp
HDRS
../remainder.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -194,8 +156,6 @@ add_entrypoint_object(
remainderf.cpp
HDRS
../remainderf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -204,8 +164,6 @@ add_entrypoint_object(
rint.cpp
HDRS
../rint.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -214,8 +172,6 @@ add_entrypoint_object(
rintf.cpp
HDRS
../rintf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -224,8 +180,6 @@ add_entrypoint_object(
round.cpp
HDRS
../round.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -234,8 +188,6 @@ add_entrypoint_object(
sqrt.cpp
HDRS
../sqrt.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -244,8 +196,6 @@ add_entrypoint_object(
sqrtf.cpp
HDRS
../sqrtf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -254,8 +204,6 @@ add_entrypoint_object(
trunc.cpp
HDRS
../trunc.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -264,8 +212,6 @@ add_entrypoint_object(
truncf.cpp
HDRS
../truncf.h
- COMPILE_OPTIONS
- -O2
)
add_entrypoint_object(
@@ -276,7 +222,6 @@ add_entrypoint_object(
../tgamma.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -287,7 +232,6 @@ add_entrypoint_object(
../tgammaf.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -298,7 +242,6 @@ add_entrypoint_object(
../lgamma.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
add_entrypoint_object(
@@ -309,5 +252,4 @@ add_entrypoint_object(
../lgamma_r.h
COMPILE_OPTIONS
${bitcode_link_flags}
- -O2
)
``````````
</details>
https://github.com/llvm/llvm-project/pull/165782
More information about the libc-commits
mailing list