Skip to content

Commit

Permalink
Remove explicit /Zi. Cmake will already add it on MSVC if Debug or Re…
Browse files Browse the repository at this point in the history
…lWithDebInfo and it conflicts with ccache: the PDB API call failed issue is due to this)
  • Loading branch information
jmarrec committed Jan 16, 2024
1 parent c8feaff commit 72adf6f
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -392,10 +392,8 @@ endif()
if(WIN32)
if(CMAKE_SIZEOF_VOID_P EQUAL 8) # 64 bit
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /OPT:REF /OPT:NOICF")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /Zi")
else()
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /OPT:REF /OPT:NOICF")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /Zi")
endif()
endif()

Expand Down

0 comments on commit 72adf6f

Please sign in to comment.