On Mon, 5 Jun 2023, James Zern wrote:
On Tue, May 30, 2023 at 5:31 AM Martin Storsjö <mar...@martin.st> wrote:For Windows, there's no publicly defined constant for checking for the i8mm extension yet. --- libavutil/aarch64/cpu.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/libavutil/aarch64/cpu.c b/libavutil/aarch64/cpu.c index ffb00f6dd2..4b97530240 100644 --- a/libavutil/aarch64/cpu.c +++ b/libavutil/aarch64/cpu.c @@ -94,6 +94,16 @@ static int detect_flags(void) return flags; } +#elif defined(_WIN32) +#include <windows.h> + +static int detect_flags(void) +{ + int flags = 0; + if (IsProcessorFeaturePresent(PF_ARM_V82_DP_INSTRUCTIONS_AVAILABLE))I think this requires a recent Windows SDK, is compatibility a concern?
Good catch, thanks! Yes, this requires a fairly recent SDK; it's been added to mingw-w64 in 2021, and in the MS WinSDK since 10.0.22000.0. So this would indeed have failed in my own fate config with an older MSVC...
It's easy to wrap this up in an #ifdef though, since the identifier is a define (both in mingw-w64 and WinSDK).
Other than that, this code is only used for aarch64, so the target audience is much smaller than generic windows code, but nevertheless, let's not break older toolchains if we don't need to.
I'll amend the patch locally with an ifdef before pushing. // Martin _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
