----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119968/#review65630 -----------------------------------------------------------
modules/ECMEnableSanitizers.cmake <https://git.reviewboard.kde.org/r/119968/#comment45863> I don't think we need this, link_libraries does this job. modules/ECMEnableSanitizers.cmake <https://git.reviewboard.kde.org/r/119968/#comment45864> Obviously asan needs to be replaced here by a proper variable depending on the saniitizer - Albert Astals Cid On ago. 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 ago. 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 > >
_______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
