All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error
@ 2017-02-27  6:07 Waldemar Brodkorb
  2017-02-27  8:29 ` Thomas Petazzoni
  2017-02-27 20:38 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Waldemar Brodkorb @ 2017-02-27  6:07 UTC (permalink / raw)
  To: buildroot

Override gcc optimization flags with -O0.

The workaround is not required for gcc6 anymore, but
gcc6 is disabled for microblaze, because it breaks startup 
of the system somehow. So the bug is not reported upstream.

Fixes:
 http://autobuild.buildroot.net/results/a318f0838a6a602046e719103ac81965c0084d52

Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
---
 package/vlc/vlc.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/package/vlc/vlc.mk b/package/vlc/vlc.mk
index 666b8d4..a523f8f 100644
--- a/package/vlc/vlc.mk
+++ b/package/vlc/vlc.mk
@@ -15,6 +15,12 @@ VLC_AUTORECONF = YES
 # Install vlc libraries in staging.
 VLC_INSTALL_STAGING = YES
 
+# gcc bug internal compiler error: in merge_overlapping_regs, at regrename.c:304
+ifeq ($(BR2_microblaze),y)
+VLC_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -O0"
+VLC_CONF_OPTS += --disable-optimizations
+endif
+
 # VLC defines two autoconf functions which are also defined by our own pkg.m4
 # from pkgconf. Unfortunately, they are defined in a different way: VLC adds
 # --enable- options, but pkg.m4 adds --with- options. To make sure we use
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error
  2017-02-27  6:07 [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error Waldemar Brodkorb
@ 2017-02-27  8:29 ` Thomas Petazzoni
  2017-02-27 20:38 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2017-02-27  8:29 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 27 Feb 2017 07:07:25 +0100, Waldemar Brodkorb wrote:
> Override gcc optimization flags with -O0.
> 
> The workaround is not required for gcc6 anymore, but
> gcc6 is disabled for microblaze, because it breaks startup 
> of the system somehow. So the bug is not reported upstream.

If it's not required for gcc6, then perhaps we should add a
BR2_TOOLCHAIN_GCC_AT_LEAST_6 condition, so that this chunk gets a
chance to be removed at some point in the future.

> +# gcc bug internal compiler error: in merge_overlapping_regs, at regrename.c:304
> +ifeq ($(BR2_microblaze),y)

I.e.:

ifeq ($(BR2_microblaze):$(BR2_TOOLCHAIN_GCC_AT_LEAST_6),y:)

> +VLC_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -O0"
> +VLC_CONF_OPTS += --disable-optimizations
> +endif

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error
  2017-02-27  6:07 [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error Waldemar Brodkorb
  2017-02-27  8:29 ` Thomas Petazzoni
@ 2017-02-27 20:38 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2017-02-27 20:38 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 27 Feb 2017 07:07:25 +0100, Waldemar Brodkorb wrote:
> Override gcc optimization flags with -O0.
> 
> The workaround is not required for gcc6 anymore, but
> gcc6 is disabled for microblaze, because it breaks startup 
> of the system somehow. So the bug is not reported upstream.
> 
> Fixes:
>  http://autobuild.buildroot.net/results/a318f0838a6a602046e719103ac81965c0084d52
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> ---
>  package/vlc/vlc.mk | 6 ++++++
>  1 file changed, 6 insertions(+)

Applied to master after tweaking the condition so that it applies only
to gcc < 6.x.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-02-27 20:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-27  6:07 [Buildroot] [PATCH] vlc: workaround microblaze gcc internal compiler error Waldemar Brodkorb
2017-02-27  8:29 ` Thomas Petazzoni
2017-02-27 20:38 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.