Commit cd8ed5208f for aom
commit cd8ed5208f6cf78bec352599d6bc084f48fd88d3
Author: James Zern <jzern@google.com>
Date: Thu Jun 18 14:59:26 2026 -0700
rdopt.c: add an assert before using value in shift
Fixes a clang-19 static analysis warnings of the form:
```
The result of left shift is undefined because the right operand is
negative
```
Bug: 474642915
Change-Id: I410f5119dc29a468b9636f8350fd79aa3b49ed5c
diff --git a/av1/encoder/rdopt.c b/av1/encoder/rdopt.c
index fcbc4dbd19..ec3d06713e 100644
--- a/av1/encoder/rdopt.c
+++ b/av1/encoder/rdopt.c
@@ -4755,6 +4755,7 @@ static AOM_FORCE_INLINE int inter_mode_search_order_independent_skip(
int skip_motion_mode = 0;
if (mbmi->partition != PARTITION_NONE) {
+ assert(ref_type > NONE_FRAME);
int skip_ref = skip_ref_frame_mask & (1 << ref_type);
if (ref_type <= ALTREF_FRAME && skip_ref) {
// Since the compound ref modes depends on the motion estimation result of