Commit 41b307ad authored by John Hubbard's avatar John Hubbard Committed by Andrii Nakryiko
Browse files

bpftool, selftests/hid/bpf: Fix 29 clang warnings 



When building either tools/bpf/bpftool, or tools/testing/selftests/hid,
(the same Makefile is used for these), clang generates many instances of
the following:

    "clang: warning: -lLLVM-17: 'linker' input unused"

Quentin points out that the LLVM version is only required in $(LIBS),
not in $(CFLAGS), so the fix is to remove it from CFLAGS.

Suggested-by: default avatarQuentin Monnet <qmo@kernel.org>
Signed-off-by: default avatarJohn Hubbard <jhubbard@nvidia.com>
Signed-off-by: default avatarAndrii Nakryiko <andrii@kernel.org>
Acked-by: default avatarQuentin Monnet <qmo@kernel.org>
Link: https://lore.kernel.org/bpf/20240505230054.13813-1-jhubbard@nvidia.com
parent e549b39a

tools/bpf/bpftool/Makefile

+1 −1
Original line number Diff line number Diff line
@@ -147,7 +147,7 @@ ifeq ($(feature-llvm),1) 
  # If LLVM is available, use it for JIT disassembly

  CFLAGS  += -DHAVE_LLVM_SUPPORT

  LLVM_CONFIG_LIB_COMPONENTS := mcdisassembler all-targets

  CFLAGS  += $(shell $(LLVM_CONFIG) --cflags --libs $(LLVM_CONFIG_LIB_COMPONENTS))

  CFLAGS  += $(shell $(LLVM_CONFIG) --cflags)

  LIBS    += $(shell $(LLVM_CONFIG) --libs $(LLVM_CONFIG_LIB_COMPONENTS))

  ifeq ($(shell $(LLVM_CONFIG) --shared-mode),static)

    LIBS += $(shell $(LLVM_CONFIG) --system-libs $(LLVM_CONFIG_LIB_COMPONENTS))