[llvm] 497fbd0 - [LLVM][Cygwin] Remove special case for CXX extensions on Cygwin. (#138328)
via llvm-commits
llvm-commits at lists.llvm.org
Sat May 3 14:33:05 PDT 2025
Author: jeremyd2019
Date: 2025-05-04T00:33:03+03:00
New Revision: 497fbd0ee0c4c50d46b5b777cdcc3a532d1dcc91
URL: https://github.com/llvm/llvm-project/commit/497fbd0ee0c4c50d46b5b777cdcc3a532d1dcc91
DIFF: https://github.com/llvm/llvm-project/commit/497fbd0ee0c4c50d46b5b777cdcc3a532d1dcc91.diff
LOG: [LLVM][Cygwin] Remove special case for CXX extensions on Cygwin. (#138328)
This is no longer necessary, and results in an inconvenient define of
`i386` on i386 Cygwin targets which breaks compiling
llvm/include/llvm/ExecutionEngine/JITLink/i386.h.
Signed-off-by: Jeremy Drake <github at jdrake.com>
Added:
Modified:
llvm/CMakeLists.txt
Removed:
################################################################################
diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index 47cbea3d0e872..e8d9ec0d6153a 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -76,14 +76,7 @@ endif()
set(CMAKE_CXX_STANDARD ${LLVM_REQUIRED_CXX_STANDARD} CACHE STRING "C++ standard to conform to")
set(CMAKE_CXX_STANDARD_REQUIRED YES)
-
-if (CYGWIN)
- # Cygwin is a bit stricter and lack things like 'strdup', 'stricmp', etc in
- # c++xx mode.
- set(CMAKE_CXX_EXTENSIONS YES)
-else()
- set(CMAKE_CXX_EXTENSIONS NO)
-endif()
+set(CMAKE_CXX_EXTENSIONS NO)
if (NOT CMAKE_BUILD_TYPE AND NOT CMAKE_CONFIGURATION_TYPES)
message(FATAL_ERROR "
More information about the llvm-commits
mailing list