[FFmpeg-cvslog] [ffmpeg] branch master updated. 47c6af7d29 avutil/avassert: always implement av_assume with av_unreachable
ffmpeg-git at ffmpeg.org
ffmpeg-git at ffmpeg.org
Thu Aug 14 22:42:40 EEST 2025
The branch, master has been updated
via 47c6af7d299c96b2e65f5f10526e0f34e00b23c8 (commit)
from 3ab9eebba7e73fbf7f4cff0f5b8c3a0d1b8ac884 (commit)
- Log -----------------------------------------------------------------
commit 47c6af7d299c96b2e65f5f10526e0f34e00b23c8
Author: Kacper MichajÅow <kasper93 at gmail.com>
AuthorDate: Fri Aug 8 00:57:18 2025 +0200
Commit: Leo Izen <leo.izen at gmail.com>
CommitDate: Thu Aug 14 19:42:13 2025 +0000
avutil/avassert: always implement av_assume with av_unreachable
One of the design goals of this macro is the ability to support static
inline predicate functions. Which is emitting warning in clang that a
function may have side-effects and the condition will be ignored. MSVC
doesn't emit warning, but also ignore predicate in such cases.
Instead of using assume builtins, implement it using unreachable. Which
solves this case for MSVC and Clang.
This reverts ea56fe60acc139a4dc7c531f2cb9f5319cef7a09, but also extends
it to MSVC, which is affected in the similar way.
Signed-off-by: Kacper MichajÅow <kasper93 at gmail.com>
diff --git a/libavutil/avassert.h b/libavutil/avassert.h
index f4f039dfb0..209c0d9d17 100644
--- a/libavutil/avassert.h
+++ b/libavutil/avassert.h
@@ -108,16 +108,10 @@ do { \
#define av_unreachable(msg) ((void)0)
#endif
-#if AV_HAS_BUILTIN(__builtin_assume)
-#define av_assume(cond) __builtin_assume(cond)
-#elif defined(_MSC_VER)
-#define av_assume(cond) __assume(cond)
-#else
#define av_assume(cond) do { \
if (!(cond)) \
av_unreachable(); \
} while (0)
#endif
-#endif
#endif /* AVUTIL_AVASSERT_H */
-----------------------------------------------------------------------
Summary of changes:
libavutil/avassert.h | 6 ------
1 file changed, 6 deletions(-)
hooks/post-receive
--
More information about the ffmpeg-cvslog
mailing list