HomePhabricator

[CMAKE] Refactor the AddCompilerFlags facilities

Authored by Fabien on Aug 5 2019, 13:32.

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

Committed
FabienAug 6 2019, 16:33
Pushed
FabienAug 6 2019, 16:33
Reviewer
Restricted Project
Differential Revision
D3802: [CMAKE] Refactor the AddCompilerFlags facilities
Parents
rABC26082355e359: Add -ftrapv to DEBUG_CXXFLAGS when --enable-debug is used
Branches
Unknown
Tags
Unknown