On Thu, Sep 12, 2013 at 06:06:23PM +0000, Loren Merritt wrote:
> On Wed, 11 Sep 2013, Hendrik Leppkes wrote:
>
> > From: Loren Merritt <[email protected]>
> >
> > Now RET checks whether it immediately follows a branch, so the programmer
> > dosen't have to keep track of that condition.
> > REP_RET is still needed manually when it's a branch target, but that's much
> > rarer.
>
> > The implementation involves lots of spurious labels, but that's ok because
> > we strip them.
>
> That's true of x264's buildsystem, but not libav yet. So import that patch
> too:
We dropped stripping executables a long time ago in e0be794.
> --- a/configure
> +++ b/configure
> @@ -1973,6 +1973,7 @@ nm_default="nm -g"
> objformat="elf"
> pkg_config_default=pkg-config
> ranlib="ranlib"
> +strip="strip"
> yasmexe="yasm"
>
> nogas=":"
> @@ -2231,6 +2232,7 @@ cc_default="${cross_prefix}${cc_default}"
> nm_default="${cross_prefix}${nm_default}"
> pkg_config_default="${cross_prefix}${pkg_config_default}"
> ranlib="${cross_prefix}${ranlib}"
> +strip="${cross_prefix}${strip}"
>
> sysinclude_default="${sysroot}/usr/include"
>
> @@ -4140,6 +4142,7 @@ AR=$ar
> ARFLAGS=$arflags
> AR_O=$ar_o
> RANLIB=$ranlib
> +STRIP=$strip
> LN_S=$ln_s
> CPPFLAGS=$CPPFLAGS
> CFLAGS=$CFLAGS
> diff --git a/library.mak b/library.mak
> index 88d33dc..58f6360 100644
> --- a/library.mak
> +++ b/library.mak
> @@ -25,6 +25,7 @@ $(SUBDIR)%-test.i: $(SUBDIR)%.c
> $(SUBDIR)x86/%.o: $(SUBDIR)x86/%.asm
> $(DEPYASM) $(YASMFLAGS) -I $(<D)/ -M -o $@ $< > $(@:.o=.d)
> $(YASM) $(YASMFLAGS) -I $(<D)/ -o $@ $<
> + -@ $(if $(STRIP), $(STRIP) -wN '..@*' $@)
Not sure what you are trying to achieve with the if, $(STRIP) should
always be set ...
If we bring back stripping, we'll need the rest of the portability
parts from e0be794 as well.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
