[systemd-devel] [PATCH 1/2] autogen: add shortcut for building with clang

[Lennart Poettering]

On Fri, 27.12.13 22:25, Thomas H.P. Andersen (phomes at gmail.com) wrote:

> From: Thomas Hindoe Paaboel Andersen <phomes at gmail.com>
> 
> For a while I have been cleaning up warnings when building with clang.
> There are currently only two sources of warnings left: Wcast-align and Wgnu.
> I am not convinced that fixing up those up is feasible so I run with them
> disabled to spot regressions. E.g. clang is a bit more strict wrt to unused
> variables with the cleanup attribute and I have fixed a number of those since.
> 
> Like the other options in autogen.sh I have a shortcut for clang as well. I use
> 'l' for llvm.
> ---
>  autogen.sh | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/autogen.sh b/autogen.sh
> index e6289b3..91df10a 100755
> --- a/autogen.sh
> +++ b/autogen.sh
> @@ -62,6 +62,9 @@ elif [ "x$1" = "xg" ]; then
>  elif [ "x$1" = "xa" ]; then
>          ./configure CFLAGS='-g -O0 -Wsuggest-attribute=pure -Wsuggest-attribute=const -ftrapv' --enable-kdbus $args
>          make clean
> +elif [ "x$1" = "xl" ]; then
> +        ./configure CC=clang CFLAGS='-g -O0 -ftrapv -Wno-cast-align -Wno-gnu' --enable-kdbus $args
> +        make clean
>  else
>          echo
>          echo "----------------------------------------------------------------"

Looks good! Please commit!

Lennart

-- 
Lennart Poettering, Red Hat