[Lldb-commits] [lldb] r184335 - The 'register' keyword is now deprecated in C++11, so ignore the warning when compiling lldb with clang since python headers commonly use the keyword.

[Matthew Sorrels]

This change causes clang 3.2/3.3 to spew out the following warning a lot:

warning: unknown warning option '-Wno-deprecated-register'; did you mean
'-Wno-deprecated'? [-Wunknown-warning-option]

Can't seem to build at all with clang 3.4 (but that may be my fault since I
had to build 3.4 and couldn't use a binary release.)

Could this be wrapped in something like:

if (CMAKE_CXX_COMPILER_ID STREQUAL "Clang" AND CLANG_VERSION_STRING
VERSION_GREATER "3.3")

Assuming this -W works in clang 3.4?




On Wed, Jun 19, 2013 at 12:06 PM, Andy Gibbs <andyg1001 at hotmail.co.uk>wrote:

> Author: andyg
> Date: Wed Jun 19 14:06:44 2013
> New Revision: 184335
>
> URL: http://llvm.org/viewvc/llvm-project?rev=184335&view=rev
> Log:
> The 'register' keyword is now deprecated in C++11, so ignore the warning
> when compiling lldb with clang since python headers commonly use the
> keyword.
>
> Modified:
>     lldb/trunk/CMakeLists.txt
>
> Modified: lldb/trunk/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/lldb/trunk/CMakeLists.txt?rev=184335&r1=184334&r2=184335&view=diff
>
> ==============================================================================
> --- lldb/trunk/CMakeLists.txt (original)
> +++ lldb/trunk/CMakeLists.txt Wed Jun 19 14:06:44 2013
> @@ -118,6 +118,12 @@ if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU"
>    )
>  endif()
>
> +if (CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
> +  add_lldb_definitions(
> +    -Wno-deprecated-register
> +  )
> +endif()
> +
>  # Disable MSVC warnings
>  if( MSVC )
>    add_lldb_definitions(
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20130619/e4eb3fed/attachment.html>