On Sun, 23 May 2010 11:05:12 +0200
Benjamin Zores <b...@geexbox.org> wrote:

> On Sun, May 23, 2010 at 5:03 AM, Brett Nash <n...@nash.id.au> wrote:
> > Hello Ben,
> >
> >        Good to see someone finally testing this.  Obviously works
> > for me, but that doesn't count ;-)
> 
> Sure :p
> 
> > Attachment didn't come through.  Try the change below first,
> > otherwise can you either put it on the web somewhere or mail it to
> > me?  Else just paste 5 lines around 8166 & 8304.
> >
> > A first guess however:
> >        There are two ldr.32 instructions, which may be the issue.
> >        The .32 is redundant, so try changing them to ldr.
> > The lines are:
> >        72 & 235 of op_blend_mask_color_neon.c
> 
> One can say you know your good damn well :p
> This was indeed the issue, without having to see the generated
> assembly. Attached (hopefully this time) is "your" patch.

I'll apply tomorrow morning.
The error was quasi-obvious from the compiler message.  I'm curious as
to why code sourcery's gcc lets it pass without comment however.

If you have time, I'd love to see any benchmarks you can generate ;-)

        Regards,
        nash

------------------------------------------------------------------------------

_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to