[PATCH] [libcxx] Refactor CMakeLists.txt handling of compile and link flags to suppress warnings.

Eric Fiselier eric at efcs.ca
Fri Nov 14 12:53:35 PST 2014


Hi danalbert, mclow.lists,

Currently we have 5 variables that are used to specify options for building libcxx
1. `LIBCXX_CXX_FEATURE_FLAGS`
2. `LIBCXX_CXX_WARNING_FLAGS`
3. `LIBCXX_CXX_REQUIRED_FLAGS`
4. `compile_flags` (in libcxx/lib)
5. `link_flags` (in libcxx/lib)

The first three all get put into `CMAKE_CXX_FLAGS`. 
This changes the way flags are handled by only using 3 different options:

1. `LIBCXX_CXX_FLAGS` - general compile and link flags.
2. `LIBCXX_COMPILE_FLAGS` - compile only flags.
3. `LIBCXX_LINK_FLAGS` - link only flags.

This patch also removes the warning about `-nostdinc++` being unused during linking.

http://reviews.llvm.org/D6277

Files:
  CMakeLists.txt
  lib/CMakeLists.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D6277.16241.patch
Type: text/x-patch
Size: 9673 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141114/7e9e153e/attachment.bin>


More information about the cfe-commits mailing list