ARM: Remove fallback to ARMv6 implementation from NEON delegate chain
authorSiarhei Siamashka <siarhei.siamashka@nokia.com>
Wed, 2 Sep 2009 16:46:47 +0000 (19:46 +0300)
committerSiarhei Siamashka <siarhei.siamashka@nokia.com>
Sun, 6 Sep 2009 21:41:21 +0000 (00:41 +0300)
This can help to fix build problems with '-mthumb' gcc option in CFLAGS.
ARMv6 optimized code can't be compiled for thumb (because of its inline
assembly) and gets automatically disabled in configure. Reference
to it from NEON optimized code resulted in linking problems.

Every ARMv6 optimized fast path function also has a better NEON
counterpart, so there is no need to fallback to ARMv6. Shorter
delegate chain should additionally result in a bit better performance.

pixman/pixman-arm-neon.c

index 3e7f566..8a2d72e 100644 (file)
@@ -2767,8 +2767,8 @@ arm_neon_fill (pixman_implementation_t *imp,
 pixman_implementation_t *
 _pixman_implementation_create_arm_neon (void)
 {
-    pixman_implementation_t *simd = _pixman_implementation_create_arm_simd ();
-    pixman_implementation_t *imp = _pixman_implementation_create (simd);
+    pixman_implementation_t *general = _pixman_implementation_create_fast_path ();
+    pixman_implementation_t *imp = _pixman_implementation_create (general);
 
     imp->composite = arm_neon_composite;
 #if 0 /* this code has some bugs */