Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(24)

Unified Diff: include/llvm/Bitcode/NaCl/NaClReaderWriter.h

Issue 940243003: PNaCl localmod mods in LLVM to 223109 (local files only) (Closed)
Patch Set: xx Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: include/llvm/Bitcode/NaCl/NaClReaderWriter.h
diff --git a/include/llvm/Bitcode/NaCl/NaClReaderWriter.h b/include/llvm/Bitcode/NaCl/NaClReaderWriter.h
index 9806d1c42b8ee447067ada3c23484e4f5f6484d3..eb3a23f3cc1d61c348442c4f1283006607286b4e 100644
--- a/include/llvm/Bitcode/NaCl/NaClReaderWriter.h
+++ b/include/llvm/Bitcode/NaCl/NaClReaderWriter.h
@@ -18,17 +18,17 @@
#include "llvm/Support/CommandLine.h"
#include "llvm/Support/ErrorOr.h"
+#include "llvm/Support/MemoryBuffer.h"
#include <string>
namespace llvm {
- class MemoryBuffer;
class LLVMContext;
class Module;
- class raw_ostream;
class NaClBitcodeHeader;
class NaClBitstreamWriter;
class StreamingMemoryObject;
+ class raw_ostream;
/// Defines the data layout used for PNaCl bitcode files. We set the
/// data layout of the module in the bitcode readers rather than in
@@ -43,10 +43,10 @@ namespace llvm {
/// \brief Defines the integer bit size used to model pointers in PNaCl.
static const unsigned PNaClIntPtrTypeBitSize = 32;
- /// getNaClLazyBitcodeModule - Read the header of the specified bitcode buffer
- /// and prepare for lazy deserialization of function bodies. If successful,
- /// takes ownership of 'buffer' and returns a non-null pointer. On
- /// error, this returns an error code and *does not* take ownership of Buffer.
+ /// Read the header of the specified bitcode buffer and prepare for lazy
+ /// deserialization of function bodies. If successful, this takes ownership
+ /// of 'Buffer' (extending its lifetime). On error, this returns an error code
+ /// and deletes Buffer.
///
/// When Verbose is non-null, more descriptive error messages are also
/// written to Verbose.
@@ -59,15 +59,14 @@ namespace llvm {
/// 3) Unreadable.
/// When AcceptSupportedOnly is true, only form 1 is allowed. When
/// AcceptSupportedOnly is false, forms 1 and 2 are allowed.
- ErrorOr<Module *> getNaClLazyBitcodeModule(MemoryBuffer *Buffer,
- LLVMContext &Context,
- raw_ostream *Verbose = nullptr,
- bool AcceptSupportedOnly = true);
-
- /// getNaClStreamedBitcodeModule - Read the header of the specified stream
- /// and prepare for lazy deserialization and streaming of function bodies.
- /// On error, this returns null, and fills in *ErrMsg with an error
- /// description if ErrMsg is non-null.
+ ErrorOr<Module *> getNaClLazyBitcodeModule(
+ std::unique_ptr<MemoryBuffer> &&Buffer, LLVMContext &Context,
+ raw_ostream *Verbose = nullptr, bool AcceptSupportedOnly = true);
+
+ /// Read the header of the specified stream and prepare for lazy
+ /// deserialization and streaming of function bodies. On error,
+ /// this returns null, and fills in *ErrMsg with an error description
+ /// if ErrMsg is non-null.
///
/// See getNaClLazyBitcodeModule for an explanation of arguments
/// Verbose, AcceptSupportedOnly.
@@ -80,20 +79,18 @@ namespace llvm {
std::string *ErrMsg = nullptr,
bool AcceptSupportedOnly = true);
- /// NaClParseBitcodeFile - Read the specified bitcode file,
- /// returning the module. This method *never* takes ownership of Buffer.
+ /// Read the bitcode file from a buffer, returning the module.
///
/// See getNaClLazyBitcodeModule for an explanation of arguments
/// Verbose, AcceptSupportedOnly.
- ErrorOr<Module *> NaClParseBitcodeFile(MemoryBuffer *Buffer,
+ ErrorOr<Module *> NaClParseBitcodeFile(MemoryBufferRef Buffer,
LLVMContext &Context,
raw_ostream *Verbose = nullptr,
bool AcceptSupportedOnly = true);
- /// NaClWriteBitcodeToFile - Write the specified module to the
- /// specified raw output stream, using PNaCl wire format. For
- /// streams where it matters, the given stream should be in "binary"
- /// mode.
+ /// Write the specified module to the specified raw output stream, using
+ /// PNaCl wire format. For streams where it matters, the given stream
+ /// should be in "binary" mode.
///
/// The AcceptSupportedOnly argument is used to decide which PNaCl versions
/// of the PNaCl bitcode to generate. There are two forms:

Powered by Google App Engine
This is Rietveld 408576698