summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJames Zern <jzern@google.com>2021-11-10 21:30:07 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2021-11-10 21:30:07 +0000
commitec80f88c5d05539c89f8bda8df571b66cab9566d (patch)
treeaac42784681aa1a2727efba1abb433886459f126
parent888bafc78d8bddb5cfc4262c93f456c812763571 (diff)
parent16333de2890ea84fff5c2d788003f4eea2ce5600 (diff)
downloadlibvpx-ec80f88c5d05539c89f8bda8df571b66cab9566d.tar
libvpx-ec80f88c5d05539c89f8bda8df571b66cab9566d.tar.gz
libvpx-ec80f88c5d05539c89f8bda8df571b66cab9566d.tar.bz2
libvpx-ec80f88c5d05539c89f8bda8df571b66cab9566d.zip
Merge changes I1425f12d,I1e9e9ffa,I6d8f676b,I92013086 into main
* changes: mem_sse2.h: loadu_uint32 -> loadu_int32 mem_sse2.h: storeu_uint32 -> storeu_int32 vp8: fix some implicit signed -> unsigned conv warnings video_source.h,ReallocImage: quiet implicit conv warning
-rw-r--r--test/video_source.h2
-rw-r--r--vp8/common/x86/bilinear_filter_sse2.c4
-rw-r--r--vp8/decoder/decodeframe.c4
-rw-r--r--vp8/encoder/encodeframe.c4
-rw-r--r--vp8/encoder/encodemv.c2
-rw-r--r--vp8/encoder/lookahead.c4
-rw-r--r--vp8/encoder/ratectrl.c2
-rw-r--r--vpx_dsp/x86/avg_pred_sse2.c6
-rw-r--r--vpx_dsp/x86/loopfilter_sse2.c16
-rw-r--r--vpx_dsp/x86/mem_sse2.h6
-rw-r--r--vpx_dsp/x86/variance_sse2.c4
-rw-r--r--vpx_ports/x86.h2
12 files changed, 28 insertions, 28 deletions
diff --git a/test/video_source.h b/test/video_source.h
index e9340f21e..2ba3f6421 100644
--- a/test/video_source.h
+++ b/test/video_source.h
@@ -191,7 +191,7 @@ class DummyVideoSource : public VideoSource {
void ReallocImage() {
vpx_img_free(img_);
img_ = vpx_img_alloc(NULL, format_, width_, height_, 32);
- raw_sz_ = ((img_->w + 31) & ~31) * img_->h * img_->bps / 8;
+ raw_sz_ = ((img_->w + 31) & ~31u) * img_->h * img_->bps / 8;
}
vpx_image_t *img_;
diff --git a/vp8/common/x86/bilinear_filter_sse2.c b/vp8/common/x86/bilinear_filter_sse2.c
index 9bf65d804..ff6cbbd68 100644
--- a/vp8/common/x86/bilinear_filter_sse2.c
+++ b/vp8/common/x86/bilinear_filter_sse2.c
@@ -313,10 +313,10 @@ static INLINE void vertical_4x4(uint16_t *src, uint8_t *dst, const int stride,
const __m128i compensated = _mm_add_epi16(sum, round_factor);
const __m128i shifted = _mm_srai_epi16(compensated, VP8_FILTER_SHIFT);
__m128i packed = _mm_packus_epi16(shifted, shifted);
- storeu_uint32(dst, _mm_cvtsi128_si32(packed));
+ storeu_int32(dst, _mm_cvtsi128_si32(packed));
packed = _mm_srli_si128(packed, 4);
dst += stride;
- storeu_uint32(dst, _mm_cvtsi128_si32(packed));
+ storeu_int32(dst, _mm_cvtsi128_si32(packed));
dst += stride;
src += 8;
}
diff --git a/vp8/decoder/decodeframe.c b/vp8/decoder/decodeframe.c
index 67c254fa1..1c1566766 100644
--- a/vp8/decoder/decodeframe.c
+++ b/vp8/decoder/decodeframe.c
@@ -872,8 +872,8 @@ static void init_frame(VP8D_COMP *pbi) {
xd->mode_info_stride = pc->mode_info_stride;
xd->corrupted = 0; /* init without corruption */
- xd->fullpixel_mask = 0xffffffff;
- if (pc->full_pixel) xd->fullpixel_mask = 0xfffffff8;
+ xd->fullpixel_mask = ~0;
+ if (pc->full_pixel) xd->fullpixel_mask = ~7;
}
int vp8_decode_frame(VP8D_COMP *pbi) {
diff --git a/vp8/encoder/encodeframe.c b/vp8/encoder/encodeframe.c
index 2f84381d2..69271f1a7 100644
--- a/vp8/encoder/encodeframe.c
+++ b/vp8/encoder/encodeframe.c
@@ -634,8 +634,8 @@ static void init_encode_frame_mb_context(VP8_COMP *cpi) {
cpi->prob_last_coded, cpi->prob_gf_coded);
}
- xd->fullpixel_mask = 0xffffffff;
- if (cm->full_pixel) xd->fullpixel_mask = 0xfffffff8;
+ xd->fullpixel_mask = ~0;
+ if (cm->full_pixel) xd->fullpixel_mask = ~7;
vp8_zero(x->coef_counts);
vp8_zero(x->ymode_count);
diff --git a/vp8/encoder/encodemv.c b/vp8/encoder/encodemv.c
index 04adf105b..ff3896539 100644
--- a/vp8/encoder/encodemv.c
+++ b/vp8/encoder/encodemv.c
@@ -160,7 +160,7 @@ static void calc_prob(vp8_prob *p, const unsigned int ct[2]) {
const unsigned int tot = ct[0] + ct[1];
if (tot) {
- const vp8_prob x = ((ct[0] * 255) / tot) & -2;
+ const vp8_prob x = ((ct[0] * 255) / tot) & ~1u;
*p = x ? x : 1;
}
}
diff --git a/vp8/encoder/lookahead.c b/vp8/encoder/lookahead.c
index 37aa9eee8..49f851d01 100644
--- a/vp8/encoder/lookahead.c
+++ b/vp8/encoder/lookahead.c
@@ -66,8 +66,8 @@ struct lookahead_ctx *vp8_lookahead_init(unsigned int width,
depth += 1;
/* Align the buffer dimensions */
- width = (width + 15) & ~15;
- height = (height + 15) & ~15;
+ width = (width + 15) & ~15u;
+ height = (height + 15) & ~15u;
/* Allocate the lookahead structures */
ctx = calloc(1, sizeof(*ctx));
diff --git a/vp8/encoder/ratectrl.c b/vp8/encoder/ratectrl.c
index 3df34009a..9cd3963e2 100644
--- a/vp8/encoder/ratectrl.c
+++ b/vp8/encoder/ratectrl.c
@@ -314,7 +314,7 @@ static void calc_iframe_target_size(VP8_COMP *cpi) {
* bandwidth per second * fraction of the initial buffer
* level
*/
- target = cpi->oxcf.starting_buffer_level / 2;
+ target = (uint64_t)cpi->oxcf.starting_buffer_level / 2;
if (target > cpi->oxcf.target_bandwidth * 3 / 2) {
target = cpi->oxcf.target_bandwidth * 3 / 2;
diff --git a/vpx_dsp/x86/avg_pred_sse2.c b/vpx_dsp/x86/avg_pred_sse2.c
index e4e1e0e7a..c6e70f744 100644
--- a/vpx_dsp/x86/avg_pred_sse2.c
+++ b/vpx_dsp/x86/avg_pred_sse2.c
@@ -46,9 +46,9 @@ void vpx_comp_avg_pred_sse2(uint8_t *comp_pred, const uint8_t *pred, int width,
r = _mm_loadu_si128((const __m128i *)ref);
ref += 16;
} else if (width == 4) {
- r = _mm_set_epi32(loadu_uint32(ref + 3 * ref_stride),
- loadu_uint32(ref + 2 * ref_stride),
- loadu_uint32(ref + ref_stride), loadu_uint32(ref));
+ r = _mm_set_epi32(loadu_int32(ref + 3 * ref_stride),
+ loadu_int32(ref + 2 * ref_stride),
+ loadu_int32(ref + ref_stride), loadu_int32(ref));
ref += 4 * ref_stride;
} else {
diff --git a/vpx_dsp/x86/loopfilter_sse2.c b/vpx_dsp/x86/loopfilter_sse2.c
index b6ff24834..347c9fdbe 100644
--- a/vpx_dsp/x86/loopfilter_sse2.c
+++ b/vpx_dsp/x86/loopfilter_sse2.c
@@ -211,21 +211,21 @@ void vpx_lpf_vertical_4_sse2(uint8_t *s, int pitch, const uint8_t *blimit,
// 00 10 20 30 01 11 21 31 02 12 22 32 03 13 23 33
ps1ps0 = _mm_unpacklo_epi8(ps1ps0, x0);
- storeu_uint32(s + 0 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
+ storeu_int32(s + 0 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
ps1ps0 = _mm_srli_si128(ps1ps0, 4);
- storeu_uint32(s + 1 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
+ storeu_int32(s + 1 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
ps1ps0 = _mm_srli_si128(ps1ps0, 4);
- storeu_uint32(s + 2 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
+ storeu_int32(s + 2 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
ps1ps0 = _mm_srli_si128(ps1ps0, 4);
- storeu_uint32(s + 3 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
+ storeu_int32(s + 3 * pitch - 2, _mm_cvtsi128_si32(ps1ps0));
- storeu_uint32(s + 4 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
+ storeu_int32(s + 4 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
qs1qs0 = _mm_srli_si128(qs1qs0, 4);
- storeu_uint32(s + 5 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
+ storeu_int32(s + 5 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
qs1qs0 = _mm_srli_si128(qs1qs0, 4);
- storeu_uint32(s + 6 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
+ storeu_int32(s + 6 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
qs1qs0 = _mm_srli_si128(qs1qs0, 4);
- storeu_uint32(s + 7 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
+ storeu_int32(s + 7 * pitch - 2, _mm_cvtsi128_si32(qs1qs0));
}
void vpx_lpf_horizontal_16_sse2(unsigned char *s, int pitch,
diff --git a/vpx_dsp/x86/mem_sse2.h b/vpx_dsp/x86/mem_sse2.h
index 258ab38e6..8b6d4d1dd 100644
--- a/vpx_dsp/x86/mem_sse2.h
+++ b/vpx_dsp/x86/mem_sse2.h
@@ -16,12 +16,12 @@
#include "./vpx_config.h"
-static INLINE void storeu_uint32(void *dst, uint32_t v) {
+static INLINE void storeu_int32(void *dst, int32_t v) {
memcpy(dst, &v, sizeof(v));
}
-static INLINE uint32_t loadu_uint32(const void *src) {
- uint32_t v;
+static INLINE int32_t loadu_int32(const void *src) {
+ int32_t v;
memcpy(&v, src, sizeof(v));
return v;
}
diff --git a/vpx_dsp/x86/variance_sse2.c b/vpx_dsp/x86/variance_sse2.c
index 37ef64eca..67645c57a 100644
--- a/vpx_dsp/x86/variance_sse2.c
+++ b/vpx_dsp/x86/variance_sse2.c
@@ -36,8 +36,8 @@ unsigned int vpx_get_mb_ss_sse2(const int16_t *src_ptr) {
}
static INLINE __m128i load4x2_sse2(const uint8_t *const p, const int stride) {
- const __m128i p0 = _mm_cvtsi32_si128(loadu_uint32(p + 0 * stride));
- const __m128i p1 = _mm_cvtsi32_si128(loadu_uint32(p + 1 * stride));
+ const __m128i p0 = _mm_cvtsi32_si128(loadu_int32(p + 0 * stride));
+ const __m128i p1 = _mm_cvtsi32_si128(loadu_int32(p + 1 * stride));
const __m128i p01 = _mm_unpacklo_epi32(p0, p1);
return _mm_unpacklo_epi8(p01, _mm_setzero_si128());
}
diff --git a/vpx_ports/x86.h b/vpx_ports/x86.h
index 4d5391b78..651ff6460 100644
--- a/vpx_ports/x86.h
+++ b/vpx_ports/x86.h
@@ -391,7 +391,7 @@ static INLINE unsigned int x87_set_double_precision(void) {
// Reserved 01B
// Double Precision (53-Bits) 10B
// Extended Precision (64-Bits) 11B
- x87_set_control_word((mode & ~0x300) | 0x200);
+ x87_set_control_word((mode & ~0x300u) | 0x200u);
return mode;
}