shithub: libvpx

Info  •  Files  •  Log  •  Branches

Remove unnecessary conditional assignment

The assignment of the variable mode_excluded in
vp9_rd_pick_inter_mode_sub8x8 takes redundant conditional jump.
This commit removes it.

Change-Id: Ie195fbe6e54ec2ade7093d562c456a2e93143704

--- a/vp9/encoder/vp9_rdopt.c

+++ b/vp9/encoder/vp9_rdopt.c

@@ -3277,13 +3277,10 @@

         vp9_is_scaled(&cm->frame_refs[second_ref_frame - 1].sf))

       continue;

-    if (comp_pred) {

-      mode_excluded = mode_excluded ? mode_excluded

-                                    : cm->reference_mode == SINGLE_REFERENCE;

-    } else if (ref_frame != INTRA_FRAME) {

-      mode_excluded = mode_excluded ? mode_excluded

-                                    : cm->reference_mode == COMPOUND_REFERENCE;

-    }

+    if (comp_pred)

+      mode_excluded = cm->reference_mode == SINGLE_REFERENCE;

+    else if (ref_frame != INTRA_FRAME)

+      mode_excluded = cm->reference_mode == COMPOUND_REFERENCE;

     // If the segment reference frame feature is enabled....

     // then do nothing if the current ref frame is not allowed..