Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Koen Kooi <k.kooi@student.utwente.nl>
To: openembedded-devel@openembedded.org
Subject: Re: Heads up: staging changes are going to land
Date: Fri, 03 Oct 2008 09:12:42 +0200	[thread overview]
Message-ID: <gc4gla$8ho$1@ger.gmane.org> (raw)
In-Reply-To: <1222986336.5190.33.camel@dax.rpnet.com>

On 03-10-2008 00:25, Richard Purdie wrote:
> Hi,
>
> On Thu, 2008-10-02 at 20:53 +0200, Koen Kooi wrote:
>> The changed described in
>> http://lists.linuxtogo.org/pipermail/openembedded-devel/2008-July/005819.html
>> are about to get committed.
>>
>> The patch can be previewed at
>> http://dominion.thruhere.net/koen/OE/staging-patch.txt
>
> I had a quick glance through that patch and you should probably rethink
> the bits below:
>
> +BUILDSDK_CPPFLAGS = "-isystem${STAGING_INCDIR}"
>   export CPPFLAGS = "${TARGET_CPPFLAGS}"
>   export TARGET_CPPFLAGS = "-isystem${STAGING_DIR_TARGET}${layout_includedir}"
>
>   export BUILD_CFLAGS = "${BUILD_CPPFLAGS} ${BUILD_OPTIMIZATION}"
> +BUILDSDK_CFLAGS = "${BUILDSDK_CPPFLAGS} ${BUILD_OPTIMIZATION}"
>   export CFLAGS = "${TARGET_CFLAGS}"
>   export TARGET_CFLAGS = "${TARGET_CPPFLAGS} ${SELECTED_OPTIMIZATION}"
>
>
> These SDK things are a bit nuts and I suspect should be removed. I added
> them to fix some problem in Poky and I think they may not be needed now
> but I'm not 100% and need to find time to check. OE probably doesn't
> want them, at least yet.

OK

>
> @@ -404,7 +414,7 @@ FULL_OPTIMIZATION = "-fexpensive-optimiz
>   ##################################################################
>
>   FULL_OPTIMIZATION = "-fexpensive-optimizations -fomit-frame-pointer -frename-registers -O2"
> -DEBUG_OPTIMIZATION = "-O -g"
> +DEBUG_OPTIMIZATION = "-O -fno-omit-frame-pointer -g"
>
>
> I think OE changed this for a reason and Poky never merged that change?
> Maybe I don't remember right though?

AIUI you need -fno-omit-frame-pointer on ARM to do profiling, right?

regards,

Koen





  reply	other threads:[~2008-10-03  7:17 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-02 18:53 Heads up: staging changes are going to land Koen Kooi
2008-10-02 22:25 ` Richard Purdie
2008-10-03  7:12   ` Koen Kooi [this message]
2008-10-03  8:19     ` Phil Blundell
2008-10-03  8:41     ` Stanislav Brabec
2008-10-03  7:23   ` Koen Kooi
2008-10-03  8:24     ` Richard Purdie
2008-10-06 19:10 ` Leon Woestenberg
2008-10-06 19:24   ` Koen Kooi
2008-10-06 19:38     ` Koen Kooi
2008-10-06 23:01       ` Leon Woestenberg
2008-10-07  8:29         ` Leon Woestenberg
2008-10-07  9:26           ` Koen Kooi

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='gc4gla$8ho$1@ger.gmane.org' \
    --to=k.kooi@student.utwente.nl \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=openembedded-devel@openembedded.org \
    /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