Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] toolchain: GCC bug 85180
Date: Mon, 28 May 2018 22:41:02 +0200	[thread overview]
Message-ID: <20180528224102.423ce611@windsurf.home> (raw)
In-Reply-To: <CANQCQpZTLyWT29JqqAYz0RsGwA-mWBNFZ_0qapkEqiG3GisEww@mail.gmail.com>

Hello,

On Mon, 28 May 2018 15:05:44 -0500, Matthew Weber wrote:

> > I tried O0/1/2 without luck.  I just checked though that they were
> > actually being used in the verbose boost output and turns out boost is
> > forcing a O3 as the last O value in the flags.  So there's a bug....
> > I'll see if I can reasonably fix that  and retest.
> >  
> 
> Looks like there is a bjam property which can be set.  Is this
> something we want tweak?  I did a quick test after applying the patch
> below and the build was successful.
> 
> --- a/package/boost/boost.mk
> +++ b/package/boost/boost.mk
> @@ -90,6 +90,7 @@ endif
> 
>  BOOST_OPTS += toolset=gcc \
>              threading=multi \
> +             optimization=off \
>              abi=$(BOOST_ABI) \
>              variant=$(if $(BR2_ENABLE_DEBUG),debug,release)
> 
> 
> I updated per the patch below (O3) and the build failed.  So it does
> look like disabling optimization works around the problem.  What's
> your thought going forward?  I can retest the other two packages to
> see if no optimization fixes them too?

Thanks for doing all those tests! Yes, it would be nice to retest if
the other packages can be built without optimizations. If that's the
case, we can simply disable optimizations for Microblaze for those few
packages. It's less annoying that having those additional dependencies
that we have to propagate to reverse dependencies.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2018-05-28 20:41 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-28 15:35 [Buildroot] [PATCH v2] toolchain: GCC bug 85180 Matt Weber
2018-05-28 19:29 ` Thomas Petazzoni
2018-05-28 19:50   ` Matthew Weber
2018-05-28 20:05     ` Matthew Weber
2018-05-28 20:41       ` Thomas Petazzoni [this message]
2018-05-28 21:41         ` Matthew Weber
2018-05-30 21:07           ` Arnout Vandecappelle
2018-05-30 21:23 ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180528224102.423ce611@windsurf.home \
    --to=thomas.petazzoni@bootlin.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox