On 10/05/2011 10:06 AM, Khem Raj wrote:
On Wed, Oct 5, 2011 at 9:48 AM, McClintock Matthew-B29882
<b29...@freescale.com> wrote:
On Wed, Oct 5, 2011 at 6:35 AM, Richard Purdie
<richard.pur...@linuxfoundation.org> wrote:
this is a gcc built-in define I wonder why you need to add it explicitly
Are you sure? Is the built-in define not __powerpc64__ ?
Further investigation:
[mattsm@right build_p5020ds-64b_release (testing $)]$
./tmp/sysroots/x86_64-linux/usr/bin/ppc64e5500-fsl-linux/powerpc64-fsl-linux-gcc
-c -E -dM empty.c | grep __p
#define __powerpc64__ 1
#define __powerpc__ 1
[mattsm@right build_p5020ds-64b_release (testing $)]$
yes I was trying it with a differently configured gcc. For linux gcc
defines __powerpc64__
and for darwin it has __ppc64__ as builtin define.
I think adding check for __powerpc64__ and __powerpc__
would be nice and can be submitted to sysprof upstream IMO
I think this would be a good fix to have in the upstream sysprof and it
would remove the CFLAGS_append from the recipe.
Sau!
-M
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
