Re: [PATCH] perf auto-dep: Speed up feature tests by building themin parallel

From: Ingo Molnar
Date: Wed Oct 02 2013 - 06:11:48 EST



* Jiri Olsa <jolsa@xxxxxxxxxx> wrote:

> > @@ -388,7 +410,7 @@ else
> > has_cplus_demangle := $(call try-cc,$(SOURCE_CPLUS_DEMANGLE),$(FLAGS_CPLUS_DEMANGLE),demangle)
> > ifeq ($(has_cplus_demangle),y)
> > EXTLIBS += -liberty
> > - CFLAGS += -DHAVE_CPLUS_DEMANGLE
> > + CFLAGS += -DHAVE_CPLUS_DEMANGLE_SUPPORT
> > else
> > msg := $(warning No bfd.h/libbfd found, install binutils-dev[el]/zlib-static to gain symbol demangling)
> > CFLAGS += -DNO_DEMANGLE
> > @@ -400,30 +422,29 @@ else
> > endif
>
> Why not add tests for
> has_bfd_iberty/has_bfd_iberty_z/has_cplus_demangle?

Yeah, did not get to that stage yet: I only converted the feature tests
that trigger on my system (which is about 80% of them). Will complete the
transition.

> We could remove the config/feature-tests.mak completely and other
> cleanup like remove try-cc.

Yeah - that will be a nice simplification!

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/