[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