From 2440801bd1e0a9b5b3388aec3dd78056365c70cf Mon Sep 17 00:00:00 2001 From: Peter Korsgaard Date: Mon, 26 Jan 2009 09:22:52 +0000 Subject: [PATCH] mplayer: add -fomit-frame-pointer for i386 This seems to be required to compile some of the inline asm. This solves the following compiler errors: h264.c: In function 'decode_cabac_residual': h264.c:6158: warning: passing argument 4 of 'decode_significance_8x8_x86' discards qualifiers from pointer target type cabac.h: In function 'get_cabac_noinline': cabac.h:513: error: can't find a register in class 'GENERAL_REGS' while reloading 'asm' cabac.h:513: error: 'asm' operand has impossible constraints While searching several weeks ago, I hit the following URL, which unfortunately currently doesn't seem to work anymore and I can't remember what the bug was about. Also see http://bugs.uclibc.org/view.php?id=4604 Signed-off-by: Markus Heidelberg --- package/multimedia/mplayer/mplayer.mk | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/package/multimedia/mplayer/mplayer.mk b/package/multimedia/mplayer/mplayer.mk index 719cc67540..b65d288a02 100644 --- a/package/multimedia/mplayer/mplayer.mk +++ b/package/multimedia/mplayer/mplayer.mk @@ -25,6 +25,11 @@ else MPLAYER_LARGEFILE:=--disable-largefiles endif +ifeq ($(BR2_i386),y) +# This seems to be required to compile some of the inline asm +MPLAYER_CFLAGS:=-fomit-frame-pointer +endif + $(DL_DIR)/$(MPLAYER_SOURCE): $(call DOWNLOAD,$(MPLAYER_SITE),$(MPLAYER_SOURCE)) @@ -38,7 +43,7 @@ $(MPLAYER_DIR)/.configured: $(MPLAYER_DIR)/.unpacked (cd $(MPLAYER_DIR); rm -rf config.cache; \ $(TARGET_CONFIGURE_OPTS) \ $(TARGET_CONFIGURE_ARGS) \ - CFLAGS="$(TARGET_CFLAGS)" \ + CFLAGS="$(TARGET_CFLAGS) $(MPLAYER_CFLAGS)" \ LDFLAGS="$(TARGET_LDFLAGS)" \ ./configure \ --prefix=/usr \ -- 2.30.2