* Make sure that the size of the type field can also control the output

  instruction pattern.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@324 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Bytecode/Writer/InstructionWriter.cpp b/lib/Bytecode/Writer/InstructionWriter.cpp
index 18140772a394e5947128074035b8dc52aac743ed..3ab882fa7b96f22a1b4d8e985f265b5d8e56b3fd 100644 (file)
--- a/lib/Bytecode/Writer/InstructionWriter.cpp
+++ b/lib/Bytecode/Writer/InstructionWriter.cpp
@@ -187,6 +187,7 @@ bool BytecodeWriter::processInstruction(const Instruction *I) {
     break;
   case Instruction::Store:
     Ty = I->getOperand(1)->getType();  // Encode the pointer type...
+    assert(Ty->isPointerType() && "Store to nonpointer type!?!?");
     break;
   default:              // Otherwise use the default behavior...
     Ty = NumOperands ? I->getOperand(0)->getType() : I->getType();
@@ -198,6 +199,11 @@ bool BytecodeWriter::processInstruction(const Instruction *I) {
   assert(Slot != -1 && "Type not available!!?!");
   Type = (unsigned)Slot;
 
+  // Make sure that we take the type number into consideration.  We don't want
+  // to overflow the field size for the instruction format we select.
+  //
+  if (Slot > MaxOpSlot) MaxOpSlot = Slot;
+
   // Handle the special case for cast...
   if (I->getOpcode() == Instruction::Cast) {
     // Cast has to encode the destination type as the second argument in the