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

Unified Diff: test/unittests/interpreter/bytecode-array-writer-unittest.cc

Issue 2493183002: [ignition] Fix more -Wsign-compare warnings. (Closed)
Patch Set: Address comment 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 side-by-side diff with in-line comments
Download patch
Index: test/unittests/interpreter/bytecode-array-writer-unittest.cc
diff --git a/test/unittests/interpreter/bytecode-array-writer-unittest.cc b/test/unittests/interpreter/bytecode-array-writer-unittest.cc
index a8dee97b5061b2b41a1cfa3de94e256f5066f697..5539fe7e2f401f6cea8a2386cc86bc69d6a5fb94 100644
--- a/test/unittests/interpreter/bytecode-array-writer-unittest.cc
+++ b/test/unittests/interpreter/bytecode-array-writer-unittest.cc
@@ -104,19 +104,19 @@ void BytecodeArrayWriterUnittest::WriteJumpLoop(Bytecode bytecode,
}
TEST_F(BytecodeArrayWriterUnittest, SimpleExample) {
- CHECK_EQ(bytecodes()->size(), 0);
+ CHECK_EQ(bytecodes()->size(), 0u);
Write(Bytecode::kStackCheck, {10, false});
- CHECK_EQ(bytecodes()->size(), 1);
+ CHECK_EQ(bytecodes()->size(), 1u);
Write(Bytecode::kLdaSmi, 127, {55, true});
- CHECK_EQ(bytecodes()->size(), 3);
+ CHECK_EQ(bytecodes()->size(), 3u);
Write(Bytecode::kLdar, Register(200).ToOperand());
- CHECK_EQ(bytecodes()->size(), 7);
+ CHECK_EQ(bytecodes()->size(), 7u);
Write(Bytecode::kReturn, {70, true});
- CHECK_EQ(bytecodes()->size(), 8);
+ CHECK_EQ(bytecodes()->size(), 8u);
static const uint8_t bytes[] = {B(StackCheck), B(LdaSmi), U8(127), B(Wide),
B(Ldar), R16(200), B(Return)};

Powered by Google App Engine
This is Rietveld 408576698