replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Flip the ownership of MCStreamer and MCTargetStreamer. 

MCStreamer now owns the target streamer. This prevents leaking the target
streamer.

llvm-svn: 192303
This commit is contained in:
Benjamin Kramer 2013-10-09 17:23:41 +00:00
parent 1cab418ce2
commit 4188293c72
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/include/llvm/MC/MCStreamer.h
View File

@ -66,11 +66,11 @@ typedef std::pair<const MCSection *, const MCExpr *> MCSectionSubPair;
/// be treated differently. Callers should always talk to a FooTargetStreamer.
class MCTargetStreamer {
protected:
OwningPtr<MCStreamer> Streamer;
MCStreamer *Streamer;
public:
virtual ~MCTargetStreamer();
void setStreamer(MCStreamer *S) { Streamer.reset(S); }
void setStreamer(MCStreamer *S) { Streamer = S; }
};
// FIXME: declared here because it is used from
@ -100,7 +100,7 @@ public:
///
class MCStreamer {
MCContext &Context;
MCTargetStreamer *TargetStreamer;
OwningPtr<MCTargetStreamer> TargetStreamer;
MCStreamer(const MCStreamer &) LLVM_DELETED_FUNCTION;
MCStreamer &operator=(const MCStreamer &) LLVM_DELETED_FUNCTION;

2
llvm/lib/Target/ARM/MCTargetDesc/ARMELFStreamer.cpp
View File

@ -305,7 +305,7 @@ private:
} // end anonymous namespace
ARMELFStreamer &ARMTargetELFStreamer::getStreamer() {
ARMELFStreamer *S = static_cast<ARMELFStreamer *>(Streamer.get());
ARMELFStreamer *S = static_cast<ARMELFStreamer *>(Streamer);
return *S;
}