[poppler] cmake/modules

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sun Oct 6 11:17:12 UTC 2019


 cmake/modules/PopplerMacros.cmake |   15 +++++++++++++++
 1 file changed, 15 insertions(+)

New commits:
commit d03f5ab76b0d0dee199b65fc801236a38830b5fc
Author: Even Rouault <even.rouault at spatialys.com>
Date:   Sun Oct 6 12:52:31 2019 +0200

    Do not override user-defined CMAKE_C[XX]_FLAGS for clang
    
    337585e3d881c2c2c9099888b09902119dc05bf8 unconditionnaly
    overrides any potential user-defined CMAKE_C[XX]_FLAGS.
    This for example breaks the GDAL oss-fuzz builds which build Poppler
    from source (see https://github.com/OSGeo/gdal/blob/master/gdal/fuzzers/build.sh#L54)
    
    So do the same as the GCC path where we save input CMAKE_C[XX]_FLAGS
    and reinject them in custom CMAKE_C{XX}_FLAGS_{build_configuration}
    The values are identical to GCC, execpt for the _DEBUG configuration
    where we remove '-O2 -fno-reorder-blocks -fno-schedule-insns -fno-inline'
    since clang does not support -fno-reorder-blocks and -fno-schedule-insns,
    so it is likely better to disable any optimization for proper debugging.

diff --git a/cmake/modules/PopplerMacros.cmake b/cmake/modules/PopplerMacros.cmake
index 5ccbc4ef..a9d20948 100644
--- a/cmake/modules/PopplerMacros.cmake
+++ b/cmake/modules/PopplerMacros.cmake
@@ -166,8 +166,23 @@ if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")
   set(DEFAULT_COMPILE_WARNINGS "${_warn}")
   set(DEFAULT_COMPILE_WARNINGS_EXTRA "${_warn} ${_warnx}")
 
+  set(_save_cxxflags "${CMAKE_CXX_FLAGS}")
   set(CMAKE_CXX_FLAGS                "-fno-exceptions -fno-check-new -fno-common -D_DEFAULT_SOURCE")
+  set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-O2 -g ${_save_cxxflags}")
+  set(CMAKE_CXX_FLAGS_RELEASE        "-O2 -DNDEBUG ${_save_cxxflags}")
+  # clang does not support -fno-reorder-blocks -fno-schedule-insns, so do not use -O2
+  set(CMAKE_CXX_FLAGS_DEBUG          "-g ${_save_cxxflags}")
+  set(CMAKE_CXX_FLAGS_DEBUGFULL      "-g3 -fno-inline ${_save_cxxflags}")
+  set(CMAKE_CXX_FLAGS_PROFILE        "-g3 -fno-inline -ftest-coverage -fprofile-arcs ${_save_cxxflags}")
+  set(_save_cflags "${CMAKE_C_FLAGS}")
   set(CMAKE_C_FLAGS                  "-std=c99 -D_DEFAULT_SOURCE")
+  set(CMAKE_C_FLAGS_RELWITHDEBINFO   "-O2 -g ${_save_cflags}")
+  set(CMAKE_C_FLAGS_RELEASE          "-O2 -DNDEBUG ${_save_cflags}")
+  # clang does not support -fno-reorder-blocks -fno-schedule-insns, so do not use -O2
+  set(CMAKE_C_FLAGS_DEBUG            "-g ${_save_cflags}")
+  set(CMAKE_C_FLAGS_DEBUGFULL        "-g3 -fno-inline ${_save_cflags}")
+  set(CMAKE_C_FLAGS_PROFILE          "-g3 -fno-inline -ftest-coverage -fprofile-arcs ${_save_cflags}")
+
 endif()
 
 if(CMAKE_C_COMPILER MATCHES "icc")


More information about the poppler mailing list