[FFmpeg-devel] [PATCH]Drop local label from ppc asm timer

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Jun 29 18:58:44 CEST 2013


Hi!

Attached patch allows compilation of ppc asm timer on aix, the aix assembler 
does not support local labels.
I am not sure how to "test" the patch, but I compared disassembly on the 
following platforms (identical with and without the patch):
ppc32 Linux
ppc64 Linux
ppc32 OSX
ppc64 OSX

Please review, Carl Eugen
-------------- next part --------------
diff --git a/libavutil/ppc/timer.h b/libavutil/ppc/timer.h
index 155fc01..9ade7f5 100644
--- a/libavutil/ppc/timer.h
+++ b/libavutil/ppc/timer.h
@@ -31,12 +31,11 @@ static inline uint64_t read_time(void)
 
      /* from section 2.2.1 of the 32-bit PowerPC PEM */
      __asm__ volatile(
-         "1:\n"
          "mftbu  %2\n"
          "mftb   %0\n"
          "mftbu  %1\n"
          "cmpw   %2,%1\n"
-         "bne    1b\n"
+         "bne    $-0x10\n"
      : "=r"(tbl), "=r"(tbu), "=r"(temp)
      :
      : "cc");


More information about the ffmpeg-devel mailing list