shithub: libvpx

Info  •  Files  •  Log  •  Branches

changed to comply with strict aliasing rule

The clamp operation may not affect the values of the final assigned mv
where compiler may make use of strict aliasing rule to optimize out the
clamp operation. This change made the code segments to better comply
the strict aliasing rule.

Change-Id: I24502ff18bd4f9e62507a879cc8760a91a0fd07e

--- a/vp8/decoder/decodemv.c

+++ b/vp8/decoder/decodemv.c

@@ -512,15 +512,15 @@

                 else

                 {

                     mbmi->mode =  NEARMV;

-                    vp8_clamp_mv2(&near_mvs[CNT_NEAR], &pbi->mb);

                     mbmi->mv.as_int = near_mvs[CNT_NEAR].as_int;

+                    vp8_clamp_mv2(&mbmi->mv, &pbi->mb);

                 }

             }

             else

             {

                 mbmi->mode =  NEARESTMV;

-                vp8_clamp_mv2(&near_mvs[CNT_NEAREST], &pbi->mb);

                 mbmi->mv.as_int = near_mvs[CNT_NEAREST].as_int;

+                vp8_clamp_mv2(&mbmi->mv, &pbi->mb);

             }

         }

         else