Add abilities to generate bitcode buffers from munged bitcode.

unittests/Bitcode/NaClAbbrevErrorTests.cpp

 //===- llvm/unittest/Bitcode/NaClAbbrevErrorTests.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 57 matching lines @@
       "      64:0|0: <65534>                   |}\n"
       "",
       DumpMunger.getTestResults());
 
   // Shows what happens when we change the abbreviation index to an
   // illegal value.
   const uint64_t AbbrevIndex4[] = {
     ReplaceIndex, NaClMungedBitcode::Replace,
     4, naclbitc::TYPE_CODE_VOID, Terminator,
   };
+
+  // Show that by default, one can't write a bad abbreviation index.
   DumpMunger.setRunAsDeathTest(true);
   EXPECT_DEATH(
       DumpMunger.runTest("Bad abbreviation index 4",
                          AbbrevIndex4, array_lengthof(AbbrevIndex4)),
+      ".*Error \\(Block 17\\)\\: Uses illegal abbreviation index\\:"
+      "        4\\: \\[2\\].*");
+
+  // Show that the corresponding error is generated when reading
+  // bitcode with a bad abbreviation index.
+  DumpMunger.WriteFlags.SaveBadAbbrevIndices = true;
+  EXPECT_DEATH(
+      DumpMunger.runTest("Bad abbreviation index 4",
+                         AbbrevIndex4, array_lengthof(AbbrevIndex4)),
       ".*Fatal\\(35\\:0\\)\\: Invalid abbreviation \\# 4 defined for record.*");
 
   // Test that bitcode reader reports problem correctly.
   NaClParseBitcodeMunger Munger(BitcodeRecords,
                                 array_lengthof(BitcodeRecords), Terminator);
+  Munger.WriteFlags.SaveBadAbbrevIndices = true;

[jvoung (off chromium), 2015/05/06 22:08:09]

It looks like there you could also separately test WriteFlags.TryRecover = true
w/ SaveBadAbbrevIndices = false, based on the check

+      if (!Flags.TryToRecover && !Flags.SaveBadAbbrevIndices)
+        return false;

[Karl, 2015/05/07 20:09:18]

Added recovery cases. Also refactored the RunAsDeathTest so that it applies to
all mungers. I did this to guarantee we can see what did occurred before the
death of the test.

I also moved death tests into their own copies, so that there is no state
pollution (since the death may leave the munger in a bad state).

   EXPECT_DEATH(
       Munger.runTest("Bad abbreviation index",
                      AbbrevIndex4, array_lengthof(AbbrevIndex4), true),
       ".*Fatal\\(35\\:0\\)\\: Invalid abbreviation \\# 4 defined for record.*");
 }
 
 } // end of anonymous namespace.