]> granicus.if.org Git - libvpx/commitdiff
vpx_convolve_copy_neon: prefer != 0 to > 0 in tests
authorJames Zern <jzern@google.com>
Wed, 28 Apr 2021 01:02:35 +0000 (18:02 -0700)
committerJames Zern <jzern@google.com>
Wed, 28 Apr 2021 01:02:35 +0000 (18:02 -0700)
this produces better assembly code

Change-Id: I80ed1a165512e941b35a4965faa0c44403357e91

vpx_dsp/arm/vpx_convolve_copy_neon.c

index 7abed67a4082258d4650d33cb9933819cfd54074..361ec8a806e917628929d20a74064820e24c51c5 100644 (file)
@@ -33,7 +33,7 @@ void vpx_convolve_copy_neon(const uint8_t *src, ptrdiff_t src_stride,
       src += src_stride;
       dst += dst_stride;
       h -= 2;
-    } while (h > 0);
+    } while (h != 0);
   } else if (w == 8) {  // copy8
     uint8x8_t s0, s1;
     do {
@@ -47,7 +47,7 @@ void vpx_convolve_copy_neon(const uint8_t *src, ptrdiff_t src_stride,
       vst1_u8(dst, s1);
       dst += dst_stride;
       h -= 2;
-    } while (h > 0);
+    } while (h != 0);
   } else if (w < 32) {  // copy16
     uint8x16_t s0, s1;
     do {
@@ -61,7 +61,7 @@ void vpx_convolve_copy_neon(const uint8_t *src, ptrdiff_t src_stride,
       vst1q_u8(dst, s1);
       dst += dst_stride;
       h -= 2;
-    } while (h > 0);
+    } while (h != 0);
   } else if (w == 32) {  // copy32
     uint8x16_t s0, s1, s2, s3;
     do {
@@ -79,7 +79,7 @@ void vpx_convolve_copy_neon(const uint8_t *src, ptrdiff_t src_stride,
       vst1q_u8(dst + 16, s3);
       dst += dst_stride;
       h -= 2;
-    } while (h > 0);
+    } while (h != 0);
   } else {  // copy64
     uint8x16_t s0, s1, s2, s3;
     do {