diff options
author | Chris Lattner <sabre@nondot.org> | 2003-12-23 20:05:15 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2003-12-23 20:05:15 +0000 |
commit | a989b234bdc848ecc220e2c932b7525b579bdfe2 (patch) | |
tree | 97bdcf0d23709f061276e929f862b15ee2a642f7 | |
parent | 3bf564a95fc5732c0e48adb4c369ac2563dabe2a (diff) |
Minor cleanups, plug a minor memory leak
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10596 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/AsmParser/ParserInternals.h | 3 | ||||
-rw-r--r-- | lib/AsmParser/llvmAsmParser.y | 4 |
2 files changed, 2 insertions, 5 deletions
diff --git a/lib/AsmParser/ParserInternals.h b/lib/AsmParser/ParserInternals.h index c18434d1f0..bb248c34ed 100644 --- a/lib/AsmParser/ParserInternals.h +++ b/lib/AsmParser/ParserInternals.h @@ -67,8 +67,7 @@ static inline void ThrowException(const std::string &message, // discriminated union. // // Note that I can't implement this class in a straight forward manner with -// constructors and stuff because it goes in a union, and GCC doesn't like -// putting classes with ctor's in unions. :( +// constructors and stuff because it goes in a union. // struct ValID { enum { diff --git a/lib/AsmParser/llvmAsmParser.y b/lib/AsmParser/llvmAsmParser.y index 5706b5396e..8049dc8c6a 100644 --- a/lib/AsmParser/llvmAsmParser.y +++ b/lib/AsmParser/llvmAsmParser.y @@ -1122,10 +1122,8 @@ ConstVal: Types '[' ConstVector ']' { // Nonempty unsized arr if (I != CurModule.GlobalRefs.end()) { V = I->second; // Placeholder already exists, use it... + $2.destroy(); } else { - // TODO: Include line number info by creating a subclass of - // TODO: GlobalVariable here that includes the said information! - // Create a placeholder for the global variable reference... GlobalVariable *GV = new GlobalVariable(PT->getElementType(), false, |