From 6ed81e253cbfb4a42ee9cf0d25d2756548a11f7d Mon Sep 17 00:00:00 2001 From: Reid Spencer Date: Thu, 27 May 2004 20:18:51 +0000 Subject: [PATCH] Use the SymbolTable::isEmpty() method instead of checking for no value planes. A SymbolTable could still have types in it! This fixes problems with two regression tests that failed because a symbol table that only contained types was being omitted from bytecode files. Thanks to Chris for the reduced test case that helped find this immediately. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@13842 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Bytecode/Writer/Writer.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/Bytecode/Writer/Writer.cpp b/lib/Bytecode/Writer/Writer.cpp index d139a6f3098..5b6fdd79004 100644 --- a/lib/Bytecode/Writer/Writer.cpp +++ b/lib/Bytecode/Writer/Writer.cpp @@ -307,7 +307,7 @@ void BytecodeWriter::outputCompactionTable() { void BytecodeWriter::outputSymbolTable(const SymbolTable &MST) { // Do not output the Bytecode block for an empty symbol table, it just wastes // space! - if (MST.plane_begin() == MST.plane_end()) return; + if ( MST.isEmpty() ) return; BytecodeBlock SymTabBlock(BytecodeFormat::SymbolTable, Out, true/* ElideIfEmpty*/);