From fdc8f785cd52a4438d5bf914594253b7787ab80e Mon Sep 17 00:00:00 2001 From: Daniel Dunbar Date: Wed, 29 Feb 2012 20:30:56 +0000 Subject: [PATCH] Bitcode: Don't expose WriteBitcodeToStream to clients. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@151747 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Bitcode/ReaderWriter.h | 4 ---- lib/Bitcode/Writer/BitcodeWriter.cpp | 5 ++--- unittests/Bitcode/BitReaderTest.cpp | 12 ++++++------ 3 files changed, 8 insertions(+), 13 deletions(-) diff --git a/include/llvm/Bitcode/ReaderWriter.h b/include/llvm/Bitcode/ReaderWriter.h index ce975cdb8bf..cc2b473f2c5 100644 --- a/include/llvm/Bitcode/ReaderWriter.h +++ b/include/llvm/Bitcode/ReaderWriter.h @@ -63,10 +63,6 @@ namespace llvm { /// should be in "binary" mode. void WriteBitcodeToFile(const Module *M, raw_ostream &Out); - /// WriteBitcodeToStream - Write the specified module to the specified - /// raw output stream. - void WriteBitcodeToStream(const Module *M, BitstreamWriter &Stream); - /// createBitcodeWriterPass - Create and return a pass that writes the module /// to the specified ostream. ModulePass *createBitcodeWriterPass(raw_ostream &Str); diff --git a/lib/Bitcode/Writer/BitcodeWriter.cpp b/lib/Bitcode/Writer/BitcodeWriter.cpp index 0e8d3acfd81..7078169c4c5 100644 --- a/lib/Bitcode/Writer/BitcodeWriter.cpp +++ b/lib/Bitcode/Writer/BitcodeWriter.cpp @@ -1823,6 +1823,7 @@ static void EmitDarwinBCTrailer(BitstreamWriter &Stream, unsigned BufferSize) { } } +static void WriteBitcodeToStream(const Module *M, BitstreamWriter &Stream); /// WriteBitcodeToFile - Write the specified module to the specified output /// stream. @@ -1838,9 +1839,7 @@ void llvm::WriteBitcodeToFile(const Module *M, raw_ostream &Out) { Out.write((char*)&Buffer.front(), Buffer.size()); } -/// WriteBitcodeToStream - Write the specified module to the specified output -/// stream. -void llvm::WriteBitcodeToStream(const Module *M, BitstreamWriter &Stream) { +static void WriteBitcodeToStream(const Module *M, BitstreamWriter &Stream) { // If this is darwin or another generic macho target, emit a file header and // trailer if needed. Triple TT(M->getTargetTriple()); diff --git a/unittests/Bitcode/BitReaderTest.cpp b/unittests/Bitcode/BitReaderTest.cpp index 91e6c151558..68cfe2836a2 100644 --- a/unittests/Bitcode/BitReaderTest.cpp +++ b/unittests/Bitcode/BitReaderTest.cpp @@ -7,6 +7,7 @@ // //===----------------------------------------------------------------------===// +#include "llvm/ADT/SmallString.h" #include "llvm/Analysis/Verifier.h" #include "llvm/Bitcode/BitstreamWriter.h" #include "llvm/Bitcode/ReaderWriter.h" @@ -43,17 +44,16 @@ static Module *makeLLVMModule() { return Mod; } -static void writeModuleToBuffer(std::vector &Buffer) { +static void writeModuleToBuffer(SmallVectorImpl &Buffer) { Module *Mod = makeLLVMModule(); - BitstreamWriter Stream(Buffer); - WriteBitcodeToStream(Mod, Stream); + raw_svector_ostream OS(Buffer); + WriteBitcodeToFile(Mod, OS); } TEST(BitReaderTest, MaterializeFunctionsForBlockAddr) { // PR11677 - std::vector Mem; + SmallString<1024> Mem; writeModuleToBuffer(Mem); - StringRef Data((const char*)&Mem[0], Mem.size()); - MemoryBuffer *Buffer = MemoryBuffer::getMemBuffer(Data, "test", false); + MemoryBuffer *Buffer = MemoryBuffer::getMemBuffer(Mem.str(), "test", false); std::string errMsg; Module *m = getLazyBitcodeModule(Buffer, getGlobalContext(), &errMsg); PassManager passes; -- 2.34.1