[FFmpeg-devel] [PATCH 5/9] SBR DSP x86: implement SSE hf_apply_noise
christophe.gisquet at gmail.com
Fri Apr 5 23:57:03 CEST 2013
2013/4/5 Michael Niedermayer <michaelni at gmx.at>:
>> +INIT_XMM sse
>> +%ifdef PIC
> i suspect you meant ifNdef
I'm confused. From what I understood from your compile error, you had
a non-PIC config (win64 is eg always PIC and does not exhibit it). In
the case of such a non-PIC config, I'm supposed to allocate another
reg and lea said address into it.
Therefore for PIC, no additional reg and directly use the address.
For non-PIC, one more and use it for addressing.
And therefore this:
>> +%define NPICREGS 0
>> +%define NOISE_TABLE sbr_noise_table
>> +%define NPICREGS 1
>> +%define NOISE_TABLE r5q
So where am I mistaken?
>> + mova m0, [kxq + ps_noise13 + 16]
> these 2 will break PIC too
OK, I'll lea the ps_noise* to the !PIC-allocated r5q and use it instead.
Jason had a proposal for AVX code. I can test the SSE changes, but not
the actual ymm stuff (nor understand it enough).
More information about the ffmpeg-devel