Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Side by Side Diff: test/unittests/wasm/leb-helper-unittest.cc

Issue 2487673004: [wasm] Fix -Wsign-compare warnings. (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "test/unittests/test-utils.h" 5 #include "test/unittests/test-utils.h"
6 6
7 #include "src/objects-inl.h" 7 #include "src/objects-inl.h"
8 #include "src/wasm/decoder.h" 8 #include "src/wasm/decoder.h"
9 #include "src/wasm/leb-helper.h" 9 #include "src/wasm/leb-helper.h"
10 10
11 namespace v8 { 11 namespace v8 {
12 namespace internal { 12 namespace internal {
13 namespace wasm { 13 namespace wasm {
14 14
15 class LEBHelperTest : public TestWithZone {}; 15 class LEBHelperTest : public TestWithZone {};
16 16
17 TEST_F(LEBHelperTest, sizeof_u32v) { 17 TEST_F(LEBHelperTest, sizeof_u32v) {
18 EXPECT_EQ(1, LEBHelper::sizeof_u32v(0)); 18 EXPECT_EQ(1, LEBHelper::sizeof_u32v(0));
19 EXPECT_EQ(1, LEBHelper::sizeof_u32v(1)); 19 EXPECT_EQ(1, LEBHelper::sizeof_u32v(1));
20 EXPECT_EQ(1, LEBHelper::sizeof_u32v(3)); 20 EXPECT_EQ(1, LEBHelper::sizeof_u32v(3));
21 21
22 for (uint32_t i = 4; i < 128; i++) { 22 for (uint32_t i = 4; i < 128; i++) {
23 EXPECT_EQ(1, LEBHelper::sizeof_u32v(i)); 23 EXPECT_EQ(1, LEBHelper::sizeof_u32v(i));
24 } 24 }
25 25
26 for (uint32_t i = (1 << 7); i < (1 << 9); i++) { 26 for (uint32_t i = (1u << 7); i < (1u << 9); i++) {
27 EXPECT_EQ(2, LEBHelper::sizeof_u32v(i)); 27 EXPECT_EQ(2, LEBHelper::sizeof_u32v(i));
28 } 28 }
29 29
30 for (uint32_t i = (1 << 14); i < (1 << 16); i += 33) { 30 for (uint32_t i = (1u << 14); i < (1u << 16); i += 33) {
31 EXPECT_EQ(3, LEBHelper::sizeof_u32v(i)); 31 EXPECT_EQ(3, LEBHelper::sizeof_u32v(i));
32 } 32 }
33 33
34 for (uint32_t i = (1 << 21); i < (1 << 24); i += 33999) { 34 for (uint32_t i = (1u << 21); i < (1u << 24); i += 33999) {
35 EXPECT_EQ(4, LEBHelper::sizeof_u32v(i)); 35 EXPECT_EQ(4, LEBHelper::sizeof_u32v(i));
36 } 36 }
37 37
38 for (uint32_t i = (1 << 28); i < (1 << 31); i += 33997779) { 38 for (uint32_t i = (1u << 28); i < (1u << 31); i += 33997779u) {
39 EXPECT_EQ(5, LEBHelper::sizeof_u32v(i)); 39 EXPECT_EQ(5, LEBHelper::sizeof_u32v(i));
40 } 40 }
41 41
42 EXPECT_EQ(5, LEBHelper::sizeof_u32v(0xFFFFFFFF)); 42 EXPECT_EQ(5, LEBHelper::sizeof_u32v(0xFFFFFFFF));
43 } 43 }
44 44
45 TEST_F(LEBHelperTest, sizeof_i32v) { 45 TEST_F(LEBHelperTest, sizeof_i32v) {
46 EXPECT_EQ(1, LEBHelper::sizeof_i32v(0)); 46 EXPECT_EQ(1, LEBHelper::sizeof_i32v(0));
47 EXPECT_EQ(1, LEBHelper::sizeof_i32v(1)); 47 EXPECT_EQ(1, LEBHelper::sizeof_i32v(1));
48 EXPECT_EQ(1, LEBHelper::sizeof_i32v(3)); 48 EXPECT_EQ(1, LEBHelper::sizeof_i32v(3));
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 CheckEncodeDecode_i64v(bit_cast<int64_t>(val)); 182 CheckEncodeDecode_i64v(bit_cast<int64_t>(val));
183 } 183 }
184 184
185 for (uint64_t val = 0xFFFFFFFFFFFFFF3B; val != 0; val = val << 1) { 185 for (uint64_t val = 0xFFFFFFFFFFFFFF3B; val != 0; val = val << 1) {
186 CheckEncodeDecode_i64v(bit_cast<int64_t>(val)); 186 CheckEncodeDecode_i64v(bit_cast<int64_t>(val));
187 } 187 }
188 } 188 }
189 } // namespace wasm 189 } // namespace wasm
190 } // namespace internal 190 } // namespace internal
191 } // namespace v8 191 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698