This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Enable -fuse-linker-plugin by default when possible, take 2

From: Paolo Bonzini <bonzini at gnu dot org>
To: Jan Hubicka <hubicka at ucw dot cz>
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, gcc-patches at gcc dot gnu dot org, aoliva at redhat dot com, rguenther at suse dot de
Date: Mon, 29 Nov 2010 18:09:06 +0100
Subject: Re: Enable -fuse-linker-plugin by default when possible, take 2
References: <20101128215916.GA8909@kam.mff.cuni.cz> <AANLkTinnsPtf=roRUDTL5r_7y-vMp9xghg1KbpDjhs2X@mail.gmail.com> <20101129152307.GC8909@kam.mff.cuni.cz> <20101129170144.GD8909@kam.mff.cuni.cz>

> +if test -f ../lto-plugin/Makefile ; then
> +  AC_MSG_CHECKING(linker plugin support)
> +  gcc_cv_lto_plugin=no
> ...
> +  if test x"$gcc_cv_lto_plugin" = xyes; then
> +    AC_DEFINE(HAVE_LTO_PLUGIN, 1,
> +  [Define if your linker supports plugin.])
> +  fi
> +  AC_MSG_RESULT($gcc_cv_lto_plugin)
> +fi

Please pull the lines in the quote above out of the if (i.e. leave
only the "ellipsis" part inside the if).

Even better would be to use AC_CACHE_CHECK.  In any case, build parts
are okay with that change.

Paolo

References:
- Enable -fuse-linker-plugin by default when possible
  - From: Jan Hubicka
- Re: Enable -fuse-linker-plugin by default when possible
  - From: H.J. Lu
- Re: Enable -fuse-linker-plugin by default when possible
  - From: Jan Hubicka
- Enable -fuse-linker-plugin by default when possible, take 2
  - From: Jan Hubicka

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]