linux-kernel - Re: [PATCH] Fix objtool with clang

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20161127161447.GA9347@krava>
Date:   Sun, 27 Nov 2016 17:14:47 +0100
From:   Jiri Olsa <jolsa@...hat.com>
To:     Peter Foley <pefoley2@...oley.com>
Cc:     linux-kernel@...r.kernel.org, acme@...hat.com, wangnan0@...wei.com
Subject: Re: [PATCH] Fix objtool with clang

On Sat, Nov 26, 2016 at 05:20:35PM -0500, Peter Foley wrote:
> Clang doesn't support multiple arguments being passed to -Wp, so split
> them.
> 
> Fixes this error:
> HOSTCC   tools/objtool/fixdep.o
> cat: tools/objtool/.fixdep.o.d: No such file or directory
> 
> Signed-off-by: Peter Foley <pefoley2@...oley.com>
> ---
>  tools/build/Build.include | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/build/Build.include b/tools/build/Build.include
> index 1dcb95e76f70..83c7be908009 100644
> --- a/tools/build/Build.include
> +++ b/tools/build/Build.include
> @@ -89,10 +89,10 @@ if_changed = $(if $(strip $(any-prereq) $(arg-check)),             \
>  # - per target C flags
>  # - per object C flags
>  # - BUILD_STR macro to allow '-D"$(variable)"' constructs
> -c_flags = -Wp,-MD,$(depfile),-MT,$@ $(CFLAGS) -D"BUILD_STR(s)=\#s" $(CFLAGS_$(basetarget).o) $(CFLAGS_$(obj))
> -cxx_flags = -Wp,-MD,$(depfile),-MT,$@ $(CXXFLAGS) -D"BUILD_STR(s)=\#s" $(CXXFLAGS_$(basetarget).o) $(CXXFLAGS_$(obj))
> +c_flags = -Wp,-MD,$(depfile) -Wp,-MT,$@ $(CFLAGS) -D"BUILD_STR(s)=\#s" $(CFLAGS_$(basetarget).o) $(CFLAGS_$(obj))
> +cxx_flags = -Wp,-MD,$(depfile) -Wp,-MT,$@ $(CXXFLAGS) -D"BUILD_STR(s)=\#s" $(CXXFLAGS_$(basetarget).o) $(CXXFLAGS_$(obj))

could you please rebase this on Arnaldo's latest perf/core branch?
(git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git)

thanks,
jirka