From a8a37a25c9e1cf11bcd6ef7958551ccbbc5976ae Mon Sep 17 00:00:00 2001 From: Wan-Teh Chang Date: Mon, 31 Jul 2023 13:00:00 -0700 Subject: [PATCH] Eliminate a common subexpression in YPixel() Save the value of a common subexpression in a local variable. Change-Id: I5724fcf341900cb2a65eb37b505194b8d3c3da9a Reviewed-on: https://chromium-review.googlesource.com/c/libyuv/libyuv/+/4735651 Reviewed-by: Frank Barchard Commit-Queue: Wan-Teh Chang --- source/row_common.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/source/row_common.cc b/source/row_common.cc index fdd49a658..7591c6b6c 100644 --- a/source/row_common.cc +++ b/source/row_common.cc @@ -1875,9 +1875,10 @@ static __inline void YPixel(uint8_t y, int yg = yuvconstants->kYToRgb[0]; #endif uint32_t y1 = (uint32_t)(y * 0x0101 * yg) >> 16; - *b = STATIC_CAST(uint8_t, Clamp(((int32_t)(y1) + ygb) >> 6)); - *g = STATIC_CAST(uint8_t, Clamp(((int32_t)(y1) + ygb) >> 6)); - *r = STATIC_CAST(uint8_t, Clamp(((int32_t)(y1) + ygb) >> 6)); + uint8_t b8 = STATIC_CAST(uint8_t, Clamp(((int32_t)(y1) + ygb) >> 6)); + *b = b8; + *g = b8; + *r = b8; } void I444ToARGBRow_C(const uint8_t* src_y,