Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Phil Blundell <pb@pbcl.net>
To: Pascal Bach <pascal.bach@siemens.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: Default value for FULL_OPTIMIZATION
Date: Tue, 02 Feb 2016 20:52:18 +0000	[thread overview]
Message-ID: <1454446338.7421.45.camel@pbcl.net> (raw)
In-Reply-To: <56B0AE88.3090103@siemens.com>

On Tue, 2016-02-02 at 14:26 +0100, Pascal Bach wrote:
> I'm not sure if the documentation is wrong or if the default set in bitbake.conf is incorrect.
> Maybe somebody can shed some light on this.

FULL_OPTIMIZATION in bitbake.conf did use to match what the
documentation says (or at least, was much closer to it).  It was changed
to the current value by commit 9cb71137 which apparently omitted to
update the documentation. 

As an aside, this commit also added "-pipe" to both FULL_OPTIMIZATION
and DEBUG_OPTIMIZATION, which seems a bit weird since (a) -pipe doesn't
have any effect on the generated code; and (b) if it's going to enabled
for both debug and release builds then it might as well go in ${CC} or
somewhere.  Also, that commit seems to have deliberately turned down the
level of optimisation somewhat in order to facilitate debugging, which
is perhaps not what folks who request FULL_OPTIMIZATION would expect.

But anyway, irrespective of the detailed merits or otherwise of that
patch, I think that updating the documentation to match what the code is
now doing would be a fine idea.

p.




  parent reply	other threads:[~2016-02-02 20:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-02 13:26 Default value for FULL_OPTIMIZATION Pascal Bach
2016-02-02 14:26 ` Mike Looijmans
2016-02-02 16:23   ` Pascal Bach
2016-02-02 14:47 ` Richard Purdie
2016-02-02 20:52 ` Phil Blundell [this message]
2016-02-03  6:17   ` Khem Raj
2016-02-03 20:48     ` Phil Blundell
2016-02-03 20:51       ` Khem Raj
2016-02-03 20:54         ` Burton, Ross
2016-02-03 20:57         ` Phil Blundell
2016-02-03 20:59           ` Khem Raj
2016-02-03 21:08             ` Phil Blundell
2016-02-03 22:37               ` Khem Raj
2016-02-04 12:56             ` Mike Looijmans

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=1454446338.7421.45.camel@pbcl.net \
    --to=pb@pbcl.net \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=pascal.bach@siemens.com \
    /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