diff --git a/lib/Bytecode/Reader/Reader.cpp b/lib/Bytecode/Reader/Reader.cpp index bff0b126aec..e735564acbd 100644 --- a/lib/Bytecode/Reader/Reader.cpp +++ b/lib/Bytecode/Reader/Reader.cpp @@ -214,6 +214,13 @@ void BytecodeParser::ParseSymbolTable(const unsigned char *&Buf, const unsigned char *EndBuf, SymbolTable *ST, Function *CurrentFunction) { + // Allow efficient basic block lookup by number. + std::vector BBMap; + if (CurrentFunction) + for (Function::iterator I = CurrentFunction->begin(), + E = CurrentFunction->end(); I != E; ++I) + BBMap.push_back(I); + while (Buf < EndBuf) { // Symtab block header: [num entries][type id number] unsigned NumEntries, Typ; @@ -223,9 +230,6 @@ void BytecodeParser::ParseSymbolTable(const unsigned char *&Buf, BCR_TRACE(3, "Plane Type: '" << *Ty << "' with " << NumEntries << " entries\n"); - Function::iterator BlockIterator; - unsigned CurBlockIteratorIdx = ~0; - for (unsigned i = 0; i < NumEntries; ++i) { // Symtab entry: [def slot #][name] unsigned slot; @@ -238,19 +242,11 @@ void BytecodeParser::ParseSymbolTable(const unsigned char *&Buf, if (Typ == Type::TypeTyID) V = (Value*)getType(slot); else if (Typ == Type::LabelTyID) { - if (!CurrentFunction) - throw std::string("Basic blocks don't exist at global scope!"); - - if (slot < CurBlockIteratorIdx) { - CurBlockIteratorIdx = 0; - BlockIterator = CurrentFunction->begin(); - } - - std::advance(BlockIterator, slot-CurBlockIteratorIdx); - CurBlockIteratorIdx = slot; - V = BlockIterator; - } else + if (slot < BBMap.size()) + V = BBMap[slot]; + } else { V = getValue(Typ, slot, false); // Find mapping... + } if (V == 0) throw std::string("Failed value look-up."); BCR_TRACE(4, "Map: '" << Name << "' to #" << slot << ":" << *V; if (!isa(V)) std::cerr << "\n");