[llvm-commits] [llvm] r134738 - in /llvm/trunk/lib/Target: ARM/AsmParser/ARMAsmParser.cpp X86/AsmParser/X86AsmParser.cpp
Benjamin Kramer
benny.kra at googlemail.com
Fri Jul 8 14:06:23 PDT 2011
Author: d0k
Date: Fri Jul 8 16:06:23 2011
New Revision: 134738
URL: http://llvm.org/viewvc/llvm-project?rev=134738&view=rev
Log:
Plug a leak by giving the AsmParser ownership of the MCSubtargetInfo.
Found by valgrind.
Modified:
llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp
Modified: llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp?rev=134738&r1=134737&r2=134738&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp (original)
+++ llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp Fri Jul 8 16:06:23 2011
@@ -25,6 +25,7 @@
#include "llvm/Target/TargetAsmParser.h"
#include "llvm/Support/SourceMgr.h"
#include "llvm/Support/raw_ostream.h"
+#include "llvm/ADT/OwningPtr.h"
#include "llvm/ADT/SmallVector.h"
#include "llvm/ADT/StringExtras.h"
#include "llvm/ADT/StringSwitch.h"
@@ -41,7 +42,7 @@
class ARMAsmParser : public TargetAsmParser {
MCAsmParser &Parser;
- const MCSubtargetInfo *STI;
+ OwningPtr<const MCSubtargetInfo> STI;
MCAsmParser &getParser() const { return Parser; }
MCAsmLexer &getLexer() const { return Parser.getLexer(); }
@@ -128,8 +129,8 @@
public:
ARMAsmParser(StringRef TT, StringRef CPU, StringRef FS, MCAsmParser &_Parser)
- : TargetAsmParser(), Parser(_Parser) {
- STI = ARM_MC::createARMMCSubtargetInfo(TT, CPU, FS);
+ : TargetAsmParser(), Parser(_Parser),
+ STI(ARM_MC::createARMMCSubtargetInfo(TT, CPU, FS)) {
MCAsmParserExtension::Initialize(_Parser);
// Initialize the set of available features.
Modified: llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp?rev=134738&r1=134737&r2=134738&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp (original)
+++ llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp Fri Jul 8 16:06:23 2011
@@ -19,6 +19,7 @@
#include "llvm/MC/MCParser/MCAsmLexer.h"
#include "llvm/MC/MCParser/MCAsmParser.h"
#include "llvm/MC/MCParser/MCParsedAsmOperand.h"
+#include "llvm/ADT/OwningPtr.h"
#include "llvm/ADT/SmallString.h"
#include "llvm/ADT/SmallVector.h"
#include "llvm/ADT/StringExtras.h"
@@ -37,7 +38,7 @@
class X86ATTAsmParser : public TargetAsmParser {
MCAsmParser &Parser;
- const MCSubtargetInfo *STI;
+ OwningPtr<const MCSubtargetInfo> STI;
private:
MCAsmParser &getParser() const { return Parser; }
@@ -79,8 +80,8 @@
public:
X86ATTAsmParser(StringRef TT, StringRef CPU, StringRef FS,
MCAsmParser &parser)
- : TargetAsmParser(), Parser(parser) {
- STI = X86_MC::createX86MCSubtargetInfo(TT, CPU, FS);
+ : TargetAsmParser(), Parser(parser),
+ STI(X86_MC::createX86MCSubtargetInfo(TT, CPU, FS)) {
// Initialize the set of available features.
setAvailableFeatures(ComputeAvailableFeatures(STI->getFeatureBits()));
More information about the llvm-commits
mailing list