diff options
| author | chriseth <chris@ethereum.org> | 2018-08-06 17:55:01 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-08-06 17:55:01 +0800 |
| commit | 74e6067347eb822c2e107c29d6ea58f22b328de7 (patch) | |
| tree | f96cebd7c011543a64b568df42606b2fb8f26601 /cmake/EthCompilerSettings.cmake | |
| parent | 3576980710e019165db272afb4fc5e80f8bd5bff (diff) | |
| parent | 6c3b48ddfcfb0849e1083cea8d4fae4446511be1 (diff) | |
| download | dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar.gz dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar.bz2 dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar.lz dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar.xz dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.tar.zst dexon-solidity-74e6067347eb822c2e107c29d6ea58f22b328de7.zip | |
Merge pull request #4602 from hackaugusto/warnings
Added guards for unknown pragmas
Diffstat (limited to 'cmake/EthCompilerSettings.cmake')
| -rw-r--r-- | cmake/EthCompilerSettings.cmake | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/cmake/EthCompilerSettings.cmake b/cmake/EthCompilerSettings.cmake index 515057fa..b0786ce2 100644 --- a/cmake/EthCompilerSettings.cmake +++ b/cmake/EthCompilerSettings.cmake @@ -36,13 +36,6 @@ if (("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU") OR ("${CMAKE_CXX_COMPILER_ID}" MA add_compile_options(-Wextra) add_compile_options(-Werror) - # Disable warnings about unknown pragmas (which is enabled by -Wall). I assume we have external - # dependencies (probably Boost) which have some of these. Whatever the case, we shouldn't be - # disabling these globally. Instead, we should pragma around just the problem #includes. - # - # TODO - Track down what breaks if we do NOT do this. - add_compile_options(-Wno-unknown-pragmas) - # Configuration-specific compiler settings. set(CMAKE_CXX_FLAGS_DEBUG "-O0 -g -DETH_DEBUG") set(CMAKE_CXX_FLAGS_MINSIZEREL "-Os -DNDEBUG") @@ -73,13 +66,13 @@ if (("${CMAKE_CXX_COMPILER_ID}" MATCHES "GNU") OR ("${CMAKE_CXX_COMPILER_ID}" MA # TODO - Is this even necessary? Why? # See http://stackoverflow.com/questions/19774778/when-is-it-necessary-to-use-use-the-flag-stdlib-libstdc. add_compile_options(-stdlib=libstdc++) - + # Tell Boost that we're using Clang's libc++. Not sure exactly why we need to do. add_definitions(-DBOOST_ASIO_HAS_CLANG_LIBCXX) - + # Use fancy colors in the compiler diagnostics add_compile_options(-fcolor-diagnostics) - + # See "How to silence unused command line argument error with clang without disabling it?" # When using -Werror with clang, it transforms "warning: argument unused during compilation" messages # into errors, which makes sense. |