Review Request 119968: add compiler sanitizers support

Alex Merry alex.merry at kde.org
Tue Sep 2 19:01:14 UTC 2014


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



modules/ECMEnableSanitizers.cmake
<https://git.reviewboard.kde.org/r/119968/#comment45951>

    More documentation. What sanitizers are supported? Provide an example cmake command. Say how to use the module (I know it's really simple, but you need to say exactly what is needed).


- Alex Merry


On Aug. 31, 2014, 6:13 p.m., Mathieu Tarral wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119968/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2014, 6:13 p.m.)
> 
> 
> Review request for Build System and Extra Cmake Modules.
> 
> 
> 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
> -----
> 
>   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/69da87b1/attachment.html>


More information about the Kde-buildsystem mailing list