On Mon, 3 Dec 2012, Justin Ruggles wrote:
> On 12/03/2012 04:19 PM, Loren Merritt wrote:
>> On Sun, 2 Dec 2012, Justin Ruggles wrote:
>>
>>> +; NOTE: This is not bit-identical with the C version because it clips to
>>> +;       [-INT_MAX, INT_MAX] instead of [INT_MIN, INT_MAX]
>>> +
>>> +INIT_XMM ssse3
>>
>> INIT_XMM ssse3, atom
>
> Why is that needed? We don't have 2 different ssse3 versions of that
> function, and that cpuflag isn't used in the function itself.

Not really needed, but I thought it was more aesthetic to have the
function name reflect what it's optimized for. Normally I expect foo_ssse3
to be an improvemenet over foo_sse2.

--Loren Merritt
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to