[Mesa-dev] [PATCH 1/3] pipe-loader: Only export necessary symbols

[Matt Turner]

On Wed, Oct 23, 2013 at 1:26 PM, Tom Stellard <tom at stellard.net> wrote:
> From: Tom Stellard <thomas.stellard at amd.com>
>
> This makes it possible to use clover with statically linked LLVM.
> ---
>  src/gallium/targets/pipe-loader/Makefile.am | 4 ++++
>  src/gallium/targets/pipe-loader/pipe.link   | 3 +++
>  2 files changed, 7 insertions(+)
>  create mode 100644 src/gallium/targets/pipe-loader/pipe.link
>
> diff --git a/src/gallium/targets/pipe-loader/Makefile.am b/src/gallium/targets/pipe-loader/Makefile.am
> index e6772b8..7091426 100644
> --- a/src/gallium/targets/pipe-loader/Makefile.am
> +++ b/src/gallium/targets/pipe-loader/Makefile.am
> @@ -22,6 +22,10 @@
>
>  include $(top_srcdir)/src/gallium/Automake.inc
>
> +LINKER_SCRIPT=pipe.link

This looks like it's a built-in variable, but it's not. Just inline
pipe.link into the line below?

> +
> +LDFLAGS+=-Wl,$(LINKER_SCRIPT)

Put spaces around the +=.

About your question -- how to force a rebuild if the linker script
changes -- with plain Make you'd list it as a dependency but with
automake I don't know. Sorry.