Re: [PATCH 1/2] perf build: Remove unused LIBPERF_CLFAGS

From: Ian Rogers
Date: Fri Mar 28 2025 - 12:30:57 EST


On Fri, Mar 28, 2025 at 5:27 AM Leo Yan <leo.yan@xxxxxxx> wrote:
>
> Since commit 91009a3a9913 ("perf build: Install libperf locally when
> building"), the LIBPERF_CFLAGS flag is never populated to libperf.
>
> As the flag is not used anymore, remove it.
>
> Signed-off-by: Leo Yan <leo.yan@xxxxxxx>

Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx>

Thanks for the cleanup!
Ian

> ---
> tools/perf/Makefile.config | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/tools/perf/Makefile.config b/tools/perf/Makefile.config
> index eea95c6c0c71..94dfc18c2741 100644
> --- a/tools/perf/Makefile.config
> +++ b/tools/perf/Makefile.config
> @@ -389,8 +389,6 @@ CORE_CFLAGS += -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -D_GNU_SOURCE
> CFLAGS += $(CORE_CFLAGS) $(INC_FLAGS)
> CXXFLAGS += $(INC_FLAGS)
>
> -LIBPERF_CFLAGS := $(CORE_CFLAGS) $(EXTRA_CFLAGS)
> -
> ifeq ($(feature-pthread-attr-setaffinity-np), 1)
> CFLAGS += -DHAVE_PTHREAD_ATTR_SETAFFINITY_NP
> endif
> --
> 2.34.1
>