Intial Thumb2 Sandbox (naclrev 6680)

src/trusted/validator_arm/validator_tests.cc

Index: src/trusted/validator_arm/validator_tests.cc
diff --git a/src/trusted/validator_arm/validator_tests.cc b/src/trusted/validator_arm/validator_tests.cc
index 0af3669903df3e334bc7fb37b7ddff535b4c0b5a..1b092396a5fa1564685576218dfd06f7df400ed2 100644
--- a/src/trusted/validator_arm/validator_tests.cc
+++ b/src/trusted/validator_arm/validator_tests.cc
@@ -191,7 +191,8 @@ TEST_F(ValidatorTests, GeneratesCorrectMasksFromSizes) {
                             kCodeRegionSize,
                             kDataRegionSize,
                             kAbiReadOnlyRegisters,
-                            kAbiDataAddrRegisters);
+                            kAbiDataAddrRegisters,
+                            0);  // ARM mode

[bsy, 2011/09/16 18:35:52]

comment like this is a good indication that an enum with symbolic names would be
nice.

   EXPECT_EQ(0xC0000000, _validator.data_address_mask())
       << "Changes in bundle size should not affect the data mask.";
   EXPECT_EQ(0xE000001F, _validator.code_address_mask())
@@ -608,7 +609,8 @@ ValidatorTests::ValidatorTests()
                kCodeRegionSize,
                kDataRegionSize,
                kAbiReadOnlyRegisters,
-               kAbiDataAddrRegisters) {}
+               kAbiDataAddrRegisters,
+               0) {}
 
 bool ValidatorTests::validate(const arm_inst *pattern,
                               size_t inst_count,