HomePhabricator

[CMAKE] Refactor the AddCompilerFlags facilities

Description

[CMAKE] Refactor the AddCompilerFlags facilities

Summary:
This adds a bit of code factorization by adding a pair of functions to
add and remove flags to target variable.
There is no behavior change.

Depends on D3800 and D3801.

Test Plan:

mkdir buildcmake && cd buildcmake
cmake -GNinja ..
ninja check

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D3802

Details

Provenance
FabienAuthored on Aug 5 2019, 13:32
nakihitoPushed on Aug 6 2019, 17:30
Reviewer
Restricted Project
Differential Revision
D3802: [CMAKE] Refactor the AddCompilerFlags facilities
Parents
rSTAGING26082355e359: Add -ftrapv to DEBUG_CXXFLAGS when --enable-debug is used
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/10648, tag: phabricator/base/10645, tag: phabricator/base/10644, tag: phabricator/base/10636