[FFmpeg-devel] [PATCH 4/5] lavu/intmath.h: Fix UB in ff_ctz_c() and ff_ctzll_c()
Rémi Denis-Courmont
remi at remlab.net
Thu May 30 16:06:41 EEST 2024
Le 30 mai 2024 12:50:05 GMT+03:00, "Tomas Härdin" <git at haerdin.se> a écrit :
>tor 2024-05-30 klockan 10:54 +0300 skrev Rémi Denis-Courmont:
>> Can't we just use the compiler built-ins here? AFAIK, they (GCC,
>> LLVM) use the same algorithm if the CPU doesn't support native CTZ.
>> And they will pick the right instruction if CPU does have CTZ.
>>
>> I get it that maybe it wasn't working so well 20 years ago, but we've
>> increased compiler version requirements since then.
>
>I think we still support MSVC, but maybe we shouldn't? It's possible to
>cross-compile for Windows either way.
I don't get how that prevents using the GCC and Clang builtins (on GCC and Clang).
More information about the ffmpeg-devel
mailing list