Review Request 119968: add compiler sanitizers support

Mathieu Tarral mathieu.tarral at gmail.com
Tue Sep 2 21:15:59 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119968/
-----------------------------------------------------------

(Updated sep. 2, 2014, 9:15 après-midi)


Review request for Build System and Extra Cmake Modules.


Changes
-------

Thank you for your different reviews.

the CMAKE_EXE_LINKER_FLAGS variable has been removed.
link_libraries() use a variable depending on the sanitizers selected.
documentation has been added to explain how to use the module.


Repository: extra-cmake-modules


Description
-------

This module uses a cached variable to hold a list of compiler sanitizers
to enable.
It parses that list, checks the sanitizer desired options, then checks the compiler
support and set the appropriate flags.


Diffs (updated)
-----

  kde-modules/KDECompilerSettings.cmake f0d1aefeabd5534944dcfff85b938f9dfa74ec19 
  modules/ECMEnableSanitizers.cmake PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/119968/diff/


Testing
-------

cmake -DECM_ENABLE_SANITIZERS=address;undefined
make VERBOSE=1


Thanks,

Mathieu Tarral

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140902/a178f4df/attachment.html>


More information about the Kde-buildsystem mailing list