Re: [bugfix 1/1] mesa-xlib: workaround gcc 4.6.0 ICE

["Kamble, Nitin A" <nitin.a.kamble@intel.com>]

> -----Original Message-----
> From: openembedded-core-bounces@lists.openembedded.org
> [mailto:openembedded-core-bounces@lists.openembedded.org] On Behalf Of
> Khem Raj
> Sent: Thursday, May 26, 2011 5:37 PM
> To: Patches and discussions about the oe-core layer
> Subject: Re: [OE-core] [bugfix 1/1] mesa-xlib: workaround gcc 4.6.0 ICE
> 
> On Thu, May 26, 2011 at 4:33 PM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > On Thu, 2011-05-26 at 13:41 -0700, nitin.a.kamble@intel.com wrote:
> >
> >> ---
> >>  meta/recipes-graphics/mesa/mesa-xlib_7.10.2.bb |    3 +++
> >>  1 files changed, 3 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/meta/recipes-graphics/mesa/mesa-xlib_7.10.2.bb
> b/meta/recipes-graphics/mesa/mesa-xlib_7.10.2.bb
> >> index b77df2c..051bd72 100644
> >> --- a/meta/recipes-graphics/mesa/mesa-xlib_7.10.2.bb
> >> +++ b/meta/recipes-graphics/mesa/mesa-xlib_7.10.2.bb
> >> @@ -17,3 +17,6 @@ PE = "1"
> >>  PR = "r0"
> >>
> >>  EXTRA_OECONF += "--with-driver=xlib"
> >> +
> >> +# nullify -O2
> >> +CFLAGS_append += " -O"
> >
> > I talked about this on IRC but simply put, no way.
> >
> > The problem is:
> >
> > a) Arm specific
> > b) determined now to be armv7 specific
> > c) gcc version specific
> >
> > and the fix should reflect this.
> >
> > So ideally when we select gcc 4.6 in tcmode-default.inc we should add
> > something there which adds a work around for mesa-xlib.
> >
> > I've suggested something like:
> >
> > TARGET_CC_ARCH_arm_pn-mesa-xlib :=
> "${@'${TARGET_CC_ARCH}'.replace('armv7','armv5')}"
> >
> > which whilst ugly, should do what we need it to.
> >
> 
> I agree with this solution however ugly it may look like. We may
> document it to explain
> the ugliness

This commit wentin yocto branch http://git.pokylinux.org/cgit/cgit.cgi/poky/commit/?id=9bccbc5fbc71b331911b558a2ba15e0a6d0046b4

With this documentation:
+# Temporary workaround for gcc 4.6.0 ICE with beagleboard
+# gcc bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47719
+TARGET_CC_ARCH_arm_pn-mesa-xlib := "${@'${TARGET_CC_ARCH}'.replace('armv7-a','armv5')}"
+

Thanks,
Nitin

> 
> > Cheers,
> >
> > Richard
> >
> >
> >
> > _______________________________________________
> > 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