[FFmpeg-cvslog] mips: inline assembly clobber list polishing

Nedeljko Babic git at videolan.org
Tue Mar 19 16:58:18 CET 2013


ffmpeg | branch: master | Nedeljko Babic <nbabic at mips.com> | Thu Feb 28 15:57:09 2013 +0100| [d23aefa4984dd79c3fac84ce160daf36b69bf3ea] | committer: Michael Niedermayer

mips: inline assembly clobber list polishing

List of clobbered registers fixed and added where it is lacking.

Signed-off-by: Nedeljko Babic <nbabic at mips.com>
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d23aefa4984dd79c3fac84ce160daf36b69bf3ea
---

 libavcodec/mips/acelp_filters_mips.c      |    3 ++-
 libavcodec/mips/acelp_vectors_mips.c      |    2 +-
 libavcodec/mips/amrwbdec_mips.c           |    2 +-
 libavcodec/mips/celp_filters_mips.c       |    8 +++++---
 libavcodec/mips/celp_math_mips.c          |    2 +-
 libavcodec/mips/compute_antialias_fixed.h |    2 ++
 libavcodec/mips/compute_antialias_float.h |    1 +
 libavcodec/mips/fft_mips.c                |    3 +++
 libavcodec/mips/lsp_mips.h                |    1 +
 libavcodec/mips/mpegaudiodsp_mips_fixed.c |   16 ++++++++++------
 libavcodec/mips/mpegaudiodsp_mips_float.c |    8 ++++++++
 11 files changed, 35 insertions(+), 13 deletions(-)

diff --git a/libavcodec/mips/acelp_filters_mips.c b/libavcodec/mips/acelp_filters_mips.c
index 1e0845c..c8d980a 100644
--- a/libavcodec/mips/acelp_filters_mips.c
+++ b/libavcodec/mips/acelp_filters_mips.c
@@ -95,6 +95,7 @@ static void ff_acelp_interpolatef_mips(float *out, const float *in,
                   [fc_val_p] "=&f" (fc_val_p), [fc_val_m] "=&f" (fc_val_m),
                   [p_filter_coeffs_m] "+r" (p_filter_coeffs_m)
                 : [prec] "r" (prec)
+                : "memory"
             );
         }
         out[n] = v;
@@ -201,7 +202,7 @@ static void ff_acelp_apply_order_2_transfer_function_mips(float *out, const floa
            [pole_coeffs] "r" (pole_coeffs)
          : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5",
            "$f6", "$f7",  "$f8", "$f9", "$f10", "$f11",
-           "$f12", "$f13", "$f14", "$f15", "$f16"
+           "$f12", "$f13", "$f14", "$f15", "$f16", "memory"
     );
 }
 #endif /* HAVE_INLINE_ASM */
diff --git a/libavcodec/mips/acelp_vectors_mips.c b/libavcodec/mips/acelp_vectors_mips.c
index e00b34e..8770df8 100644
--- a/libavcodec/mips/acelp_vectors_mips.c
+++ b/libavcodec/mips/acelp_vectors_mips.c
@@ -88,7 +88,7 @@ static void ff_weighted_vector_sumf_mips(
         : [weight_coeff_a] "f" (weight_coeff_a),
           [weight_coeff_b] "f" (weight_coeff_b),
           [length] "r" (length), [a_end]"r"(a_end)
-        : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5"
+        : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5", "memory"
     );
 }
 #endif /* HAVE_INLINE_ASM */
diff --git a/libavcodec/mips/amrwbdec_mips.c b/libavcodec/mips/amrwbdec_mips.c
index 77fae6c..1d6ed2d 100644
--- a/libavcodec/mips/amrwbdec_mips.c
+++ b/libavcodec/mips/amrwbdec_mips.c
@@ -178,7 +178,7 @@ void hb_fir_filter_mips(float *out, const float fir_coef[HB_FIR_SIZE + 1],
 
             : [output]"=&f"(output)
             : [fir_coef]"r"(fir_coef), [p_data]"r"(p_data)
-            : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5"
+            : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5", "memory"
         );
         out[i] = output;
     }
diff --git a/libavcodec/mips/celp_filters_mips.c b/libavcodec/mips/celp_filters_mips.c
index 5ecde00..ef5b07b 100644
--- a/libavcodec/mips/celp_filters_mips.c
+++ b/libavcodec/mips/celp_filters_mips.c
@@ -111,7 +111,7 @@ static void ff_celp_lp_synthesis_filterf_mips(float *out,
             : [old_out0]"f"(old_out0), [old_out1]"f"(old_out1),
               [old_out2]"f"(old_out2), [old_out3]"f"(old_out3),
               [filter_coeffs]"r"(filter_coeffs)
-            : "$f0", "$f1", "$f2", "$f3", "$f4"
+            : "$f0", "$f1", "$f2", "$f3", "$f4", "memory"
         );
 
         for (i = 5; i <= filter_length; i += 2) {
@@ -138,7 +138,7 @@ static void ff_celp_lp_synthesis_filterf_mips(float *out,
                   [old_out3]"+f"(old_out3),[p_filter_coeffs]"+r"(p_filter_coeffs),
                   [p_out]"+r"(p_out)
                 :
-                : "$f4", "$f5"
+                : "$f4", "$f5", "memory"
             );
             FFSWAP(float, old_out0, old_out2);
         }
@@ -188,6 +188,8 @@ static void ff_celp_lp_synthesis_filterf_mips(float *out,
                 : [fc_val]"=&f"(fc_val), [out_val]"+f"(out_val),
                   [out_val_i]"=&f"(out_val_i), [p_out]"+r"(p_out),
                   [p_filter_coeffs]"+r"(p_filter_coeffs)
+                :
+                : "memory"
             );
         }
         out[n] = out_val;
@@ -262,7 +264,7 @@ static void ff_celp_lp_zero_synthesis_filterf_mips(float *out,
               [fc_val]"=&f"(fc_val), [p_filter_coeffs]"+r"(p_filter_coeffs),
               [p_in]"+r"(p_in), [i]"+r"(i)
             :
-            : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5", "$f6", "$f7"
+            : "$f0", "$f1", "$f2", "$f3", "$f4", "$f5", "$f6", "$f7", "memory"
         );
 
         out[n+7] = sum_out8;
diff --git a/libavcodec/mips/celp_math_mips.c b/libavcodec/mips/celp_math_mips.c
index f0335c9..d7ccc23 100644
--- a/libavcodec/mips/celp_math_mips.c
+++ b/libavcodec/mips/celp_math_mips.c
@@ -75,7 +75,7 @@ static float ff_dot_productf_mips(const float* a, const float* b,
 
         : [sum] "=&f" (sum), [a] "+r" (a), [b] "+r" (b)
         : [a_end]"r"(a_end), [length] "r" (length)
-        : "$f1", "$f2"
+        : "$f1", "$f2", "memory"
     );
     return sum;
 }
diff --git a/libavcodec/mips/compute_antialias_fixed.h b/libavcodec/mips/compute_antialias_fixed.h
index 13684f6..a967f67 100644
--- a/libavcodec/mips/compute_antialias_fixed.h
+++ b/libavcodec/mips/compute_antialias_fixed.h
@@ -237,6 +237,8 @@ static void compute_antialias_mips_fixed(MPADecodeContext *s,
               [temp_reg5] "=&r" (temp_reg5), [temp_reg6] "=&r" (temp_reg6)
             : [csa] "r" (csa), [ptr] "r" (ptr),
               [MAX_lo] "r" (MAX_lo)
+            : "memory", "hi", "lo", "$ac1hi", "$ac1lo", "$ac2hi", "$ac2lo",
+              "$ac3hi", "$ac3lo"
          );
 
         ptr += 18;
diff --git a/libavcodec/mips/compute_antialias_float.h b/libavcodec/mips/compute_antialias_float.h
index 289150d..0f6f03f 100644
--- a/libavcodec/mips/compute_antialias_float.h
+++ b/libavcodec/mips/compute_antialias_float.h
@@ -176,6 +176,7 @@ static void compute_antialias_mips_float(MPADecodeContext *s,
           [out1] "=&f" (out1), [out2] "=&f" (out2),
           [out3] "=&f" (out3), [out4] "=&f" (out4)
         : [csa] "r" (csa), [ptr_end] "r" (ptr_end)
+        : "memory"
     );
 }
 #define compute_antialias compute_antialias_mips_float
diff --git a/libavcodec/mips/fft_mips.c b/libavcodec/mips/fft_mips.c
index 8a2d86e..02f4cb6 100644
--- a/libavcodec/mips/fft_mips.c
+++ b/libavcodec/mips/fft_mips.c
@@ -408,6 +408,8 @@ static void ff_imdct_half_mips(FFTContext *s, FFTSample *output, const FFTSample
               [tsin1]"+r"(tsin1), [tcos1]"+r"(tcos1),
               [in1]"+r"(in1), [in2]"+r"(in2),
               [in3]"+r"(in3), [in4]"+r"(in4)
+            :
+            : "memory"
         );
 
         z[j ].re = temp9;
@@ -472,6 +474,7 @@ static void ff_imdct_half_mips(FFTContext *s, FFTSample *output, const FFTSample
             : [z1]"r"(z1), [z2]"r"(z2),
               [tsin1]"r"(tsin1), [tcos1]"r"(tcos1),
               [tsin2]"r"(tsin2), [tcos2]"r"(tcos2)
+            : "memory"
         );
 
         z1[1].re = temp9;
diff --git a/libavcodec/mips/lsp_mips.h b/libavcodec/mips/lsp_mips.h
index 7497706..c55ef03 100644
--- a/libavcodec/mips/lsp_mips.h
+++ b/libavcodec/mips/lsp_mips.h
@@ -99,6 +99,7 @@ static av_always_inline void ff_lsp2polyf_mips(const double *lsp, double *f, int
               [tmp]"=&f"(tmp), [f_j]"=&f"(f_j), [p_f]"+r"(p_f),
               [j]"+r"(j), [p_fi]"+r"(p_fi)
             : [i]"r"(i)
+            : "memory"
         );
         f[1] += val;
     }
diff --git a/libavcodec/mips/mpegaudiodsp_mips_fixed.c b/libavcodec/mips/mpegaudiodsp_mips_fixed.c
index d1b0277..047a833 100644
--- a/libavcodec/mips/mpegaudiodsp_mips_fixed.c
+++ b/libavcodec/mips/mpegaudiodsp_mips_fixed.c
@@ -1,4 +1,4 @@
-/*
+    /*
  * Copyright (c) 2012
  *      MIPS Technologies, Inc., California.
  *
@@ -166,7 +166,7 @@ static void ff_mpadsp_apply_window_mips_fixed(int32_t *synth_buf, int32_t *windo
           [sum1] "+r" (sum1), [w] "+r" (w), [temp3] "+r" (temp3)
         : [p] "r" (p), [samples] "r" (samples), [min_asm] "r" (min_asm),
           [max_asm] "r" (max_asm)
-        : "hi","lo"
+        : "memory", "hi","lo"
      );
 
      samples += incr;
@@ -290,7 +290,7 @@ static void ff_mpadsp_apply_window_mips_fixed(int32_t *synth_buf, int32_t *windo
               [w] "+r" (w), [w2] "+r" (w2), [samples] "+r" (samples),
               [samples2] "+r" (samples2), [temp3] "+r" (temp3)
             : [min_asm] "r" (min_asm), [max_asm] "r" (max_asm)
-            : "hi", "lo"
+            : "memory", "hi", "lo", "$ac1hi", "$ac1lo"
         );
 
         samples += incr;
@@ -340,7 +340,7 @@ static void ff_mpadsp_apply_window_mips_fixed(int32_t *synth_buf, int32_t *windo
           [w_asm2] "=&r" (w_asm2), [p_asm2] "=&r" (p_asm2), [sum1] "+r" (sum1)
         : [w] "r" (w), [p] "r" (p), [samples] "r" (samples), [min_asm] "r" (min_asm),
           [max_asm] "r" (max_asm)
-        : "hi", "lo"
+        : "memory", "hi", "lo", "$ac1hi", "$ac1lo"
      );
 
     *dither_state= temp1;
@@ -437,6 +437,8 @@ static void imdct36_mips_fixed(int *out, int *buf, int *in, int *win)
         : [in] "+r" (in), [t1] "=&r" (t1), [t2] "=&r" (t2), [t3] "=&r" (t3),
           [t4] "=&r" (t4), [t5] "=&r" (t5), [t6] "=&r" (t6),
           [t7] "=&r" (t7), [t8] "=&r" (t8)
+        :
+        : "memory"
     );
 
     for(j = 0; j < 2; j++) {
@@ -573,7 +575,8 @@ static void imdct36_mips_fixed(int *out, int *buf, int *in, int *win)
             : [C_2] "r" (C_2), [in1] "r" (in1), [tmp1] "r" (tmp1), [C_8] "r" (C_8),
               [C_4] "r" (C_4), [C_3] "r" (C_3), [C_1] "r" (C_1), [C_7] "r" (C_7),
               [C_3A] "r" (C_3A), [C_5] "r" (C_5)
-            : "hi", "lo"
+            : "memory", "hi", "lo", "$ac1hi", "$ac1lo", "$ac2hi", "$ac2lo",
+              "$ac3hi", "$ac3lo"
          );
     }
 
@@ -873,7 +876,8 @@ static void imdct36_mips_fixed(int *out, int *buf, int *in, int *win)
           [temp_reg5] "=&r" (temp_reg5), [temp_reg6] "=&r" (temp_reg6),
           [out] "+r" (out)
         : [tmp] "r" (tmp), [win] "r" (win), [buf] "r" (buf)
-        : "hi", "lo"
+        : "memory", "hi", "lo", "$ac1hi", "$ac1lo", "$ac2hi", "$ac2lo",
+          "$ac3hi", "$ac3lo"
     );
 }
 
diff --git a/libavcodec/mips/mpegaudiodsp_mips_float.c b/libavcodec/mips/mpegaudiodsp_mips_float.c
index 00a7f20..beb8e78 100644
--- a/libavcodec/mips/mpegaudiodsp_mips_float.c
+++ b/libavcodec/mips/mpegaudiodsp_mips_float.c
@@ -273,6 +273,7 @@ static void ff_mpadsp_apply_window_mips_float(float *synth_buf, float *window,
           [t_sample] "=&r" (t_sample)
         : [synth_buf] "r" (synth_buf), [window] "r" (window),
           [dither_state] "r" (dither_state), [incr1] "r" (incr1)
+        : "memory"
     );
 }
 
@@ -348,6 +349,7 @@ static void ff_dct32_mips_float(float *out, const float *tab)
           [val16] "=f" (val16), [val23] "=f" (val23),
           [val24] "=f" (val24), [val31] "=f" (val31)
         : [tab] "r" (tab)
+        : "memory"
     );
 
     __asm__ volatile (
@@ -410,6 +412,7 @@ static void ff_dct32_mips_float(float *out, const float *tab)
           [val19] "=f" (val19), [val20] "=f" (val20),
           [val27] "=f" (val27), [val28] "=f" (val28)
         : [tab] "r" (tab)
+        : "memory"
     );
 
     __asm__ volatile (
@@ -517,6 +520,7 @@ static void ff_dct32_mips_float(float *out, const float *tab)
           [val17] "=f" (val17), [val22] "=f" (val22),
           [val25] "=f" (val25), [val30] "=f" (val30)
         : [tab] "r" (tab)
+        : "memory"
     );
 
     __asm__ volatile (
@@ -579,6 +583,7 @@ static void ff_dct32_mips_float(float *out, const float *tab)
           [val18] "=f" (val18), [val21] "=f" (val21),
           [val26] "=f" (val26), [val29] "=f" (val29)
         : [tab] "r" (tab)
+        : "memory"
     );
 
     __asm__ volatile (
@@ -868,6 +873,7 @@ static void imdct36_mips_float(float *out, float *buf, float *in, float *win)
           [out3] "=&f" (out3), [out4] "=&f" (out4),
           [out5] "=&f" (out5)
         : [in] "r" (in)
+        : "memory"
     );
 
     /* loop 3 */
@@ -999,6 +1005,7 @@ static void imdct36_mips_float(float *out, float *buf, float *in, float *win)
           [c7] "=&f" (c7), [c8] "=&f" (c8),
           [c9] "=&f" (c9)
         : [in] "r" (in), [tmp] "r" (tmp)
+        : "memory"
     );
 
     /* loop 4 */
@@ -1212,6 +1219,7 @@ static void imdct36_mips_float(float *out, float *buf, float *in, float *win)
           [s2] "=&f" (s2), [s3] "=&f" (s3)
         : [tmp] "r" (tmp), [win] "r" (win),
           [buf] "r" (buf), [out] "r" (out)
+        : "memory"
     );
 }
 



More information about the ffmpeg-cvslog mailing list