Additional clean ups on errors in bitcode parsing.

unittests/Bitcode/NaClParseInstsTest.cpp

 //===- llvm/unittest/Bitcode/NaClParseInstsTest.cpp ----------------------===//
 //     Tests parser for PNaCl bitcode instructions.
 //
 //                     The LLVM Compiler Infrastructure
 //
 // This file is distributed under the University of Illinois Open Source
 // License. See LICENSE.TXT for details.
 //
 //===----------------------------------------------------------------------===//
 
@@ skipping 72 matching lines @@
       "Error(66:4): Invalid relative value id: 100 (Must be <= 4)\n"
       "      72:6|    3: <10>                  |    ret void;\n"
       "      74:4|  0: <65534>                 |  }\n"
       "      76:0|0: <65534>                   |}\n",
       DumpMunger.getTestResults());
 
   NaClParseBitcodeMunger Munger(BitcodeRecords,
                                 array_lengthof(BitcodeRecords), Terminator);
   EXPECT_FALSE(Munger.runTest("Nonexistant call arg", true));
   EXPECT_EQ(
-      "Error: (72:6) Invalid call argument: Index 1\n"
+      "Error(72:6): Invalid call argument: Index 1\n"
       "Error: Invalid value in record\n",
       Munger.getTestResults());
 }
 
 /// Test how we recognize alignments in alloca instructions.
 TEST(NaClParseInstsTests, BadAllocaAlignment) {
   const uint64_t BitcodeRecords[] = {
     1, naclbitc::BLK_CODE_ENTER, naclbitc::MODULE_BLOCK_ID, 2, Terminator,
     1, naclbitc::BLK_CODE_ENTER, naclbitc::TYPE_BLOCK_ID_NEW, 2, Terminator,
     3, naclbitc::TYPE_CODE_NUMENTRY, 4, Terminator,
@@ skipping 62 matching lines @@
       DumpMunger.getLinesWithSubstring("alloca"));
 
   // Show what happens when changing alignment to 2**30.
   const uint64_t Align30[] = {
     ReplaceIndex, NaClBitcodeMunger::Replace,
     3, naclbitc::FUNC_CODE_INST_ALLOCA, 1, getEncAlignPower(30), Terminator,
   };
   EXPECT_FALSE(Munger.runTest(
       "BadAllocaAlignment-30", Align30, array_lengthof(Align30), true));
   EXPECT_EQ(
-      "Error: (65:6) Alignment can't be greater than 2**29. Found: 2**30\n"
+      "Error(65:6): Alignment can't be greater than 2**29. Found: 2**30\n"
       "Error: Invalid value in record\n",
       Munger.getTestResults());
   EXPECT_FALSE(DumpMunger.runTestForAssembly(
       "BadAllocaAlignment-30", Align30, array_lengthof(Align30)));
   EXPECT_EQ(
       "    %v0 = alloca i8, i32 %p0, align 0;\n",
       DumpMunger.getLinesWithSubstring("alloca"));
   EXPECT_EQ(
       "Error(62:4): Alignment can't be greater than 2**29. Found: 2**30\n",
       DumpMunger.getLinesWithSubstring("Error"));
@@ skipping 115 matching lines @@
       DumpMunger.getLinesWithSubstring("load"));
 
   // Show what happens when changing alignment to 2**30.
   const uint64_t Align30[] = {
     ReplaceIndex, NaClBitcodeMunger::Replace,
     3, naclbitc::FUNC_CODE_INST_LOAD, 1, getEncAlignPower(30), 0, Terminator,
   };
   EXPECT_FALSE(Munger.runTest(
       "BadLoadAlignment-30", Align30, array_lengthof(Align30), true));
   EXPECT_EQ(
-      "Error: (62:4) Alignment can't be greater than 2**29. Found: 2**30\n"
+      "Error(62:4): Alignment can't be greater than 2**29. Found: 2**30\n"
       "Error: Invalid value in record\n",
       Munger.getTestResults());
   EXPECT_FALSE(DumpMunger.runTestForAssembly(
       "BadLoadAlignment-30", Align30, array_lengthof(Align30)));
   EXPECT_EQ(
       "    %v0 = load i32* %p0, align 0;\n"
       "Error(58:4): load: Illegal alignment for i32. Expects: 1\n",
       DumpMunger.getLinesWithSubstring("load"));
 }
 
@@ skipping 111 matching lines @@
       DumpMunger.getLinesWithSubstring("store"));
 
   // Show what happens when changing alignment to 2**30.
   const uint64_t Align30[] = {
     ReplaceIndex, NaClBitcodeMunger::Replace,
     3, naclbitc::FUNC_CODE_INST_STORE, 2, 1, getEncAlignPower(30), Terminator,
   };
   EXPECT_FALSE(Munger.runTest(
       "BadStoreAlignment-30", Align30, array_lengthof(Align30), true));
   EXPECT_EQ(
-      "Error: (66:4) Alignment can't be greater than 2**29. Found: 2**30\n"
+      "Error(66:4): Alignment can't be greater than 2**29. Found: 2**30\n"
       "Error: Invalid value in record\n",
       Munger.getTestResults());
   EXPECT_FALSE(DumpMunger.runTestForAssembly(
       "BadStoreAlignment-30", Align30, array_lengthof(Align30)));
   EXPECT_EQ(
       "    store float %p1, float* %p0, align 0;\n"
       "Error(62:4): store: Illegal alignment for float. Expects: 1 or 4\n",
       DumpMunger.getLinesWithSubstring("store"));
 }
 
 } // end of anonamous namespace.