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. > The flags in the commit message are needed: > -mfloat-abi=softfp -mfpu=neon > Also make sure you set the appropriate CPU[1] > -mcpu=cortex-a8 They obviously were used :-) Thx, Ben
diff -Naur evas-r49141.orig/src/lib/engines/common/evas_op_blend/op_blend_mask_color_neon.c evas-r49141/src/lib/engines/common/evas_op_blend/op_blend_mask_color_neon.c --- evas-r49141.orig/src/lib/engines/common/evas_op_blend/op_blend_mask_color_neon.c 2010-05-23 10:55:16.717491714 +0200 +++ evas-r49141/src/lib/engines/common/evas_op_blend/op_blend_mask_color_neon.c 2010-05-23 10:56:37.447490792 +0200 @@ -69,7 +69,7 @@ " ble "AP"loopout \n\t" AP"quadloopint: \n\t" // " vld1.32 d0[0], [%[m]]! \n\t" - " ldr.32 %[x], [%[m]] \n\t" + " ldr %[x], [%[m]] \n\t" " add %[m], #4 \n\t" " cmp %[x], #0 \n\t" " beq "AP"fastloop \n\t" @@ -232,7 +232,7 @@ AP"quadloopint: \n\t" // Load the mask: 4 bytes: It has d0/d1 - " ldr.32 %[x], [%[m]] \n\t" + " ldr %[x], [%[m]] \n\t" " add %[m], #4 \n\t" " cmp %[x], #0 \n\t" " beq "AP"fastloop \n\t"
------------------------------------------------------------------------------
_______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
