6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-02-07 14:33:15 +00:00
Code Issues Projects Releases Wiki Activity

MipsAsmParser: Fix a couple of string use-after-frees and misuses of classof.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@163383 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Benjamin Kramer 2012-09-07 09:47:42 +00:00
parent c37290e578
commit 38539ebc2b
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/Target/Mips/AsmParser/MipsAsmParser.cpp
View File

@ -523,8 +523,7 @@ bool MipsAsmParser::ParseOperand(SmallVectorImpl<MCParsedAsmOperand*>&Operands,
SMLoc E = SMLoc::getFromPointer(Parser.getTok().getLoc().getPointer() - 1); SMLoc E = SMLoc::getFromPointer(Parser.getTok().getLoc().getPointer() - 1);
StringRef Id = StringRef("$" + Identifier.str()); MCSymbol *Sym = getContext().GetOrCreateSymbol("$" + Identifier);
MCSymbol *Sym = getContext().GetOrCreateSymbol(Id);
// Otherwise create a symbol ref. // Otherwise create a symbol ref.
const MCExpr *Res = MCSymbolRefExpr::Create(Sym, MCSymbolRefExpr::VK_None, const MCExpr *Res = MCSymbolRefExpr::Create(Sym, MCSymbolRefExpr::VK_None,
@ -571,7 +570,7 @@ bool MipsAsmParser::parseRelocOperand(const MCExpr *&Res) {
if (Tok.isNot(AsmToken::Identifier)) if (Tok.isNot(AsmToken::Identifier))
return true; return true;
StringRef Str = Tok.getIdentifier(); std::string Str = Tok.getIdentifier().str();
Parser.Lex(); //eat identifier Parser.Lex(); //eat identifier
//now make expression from the rest of the operand //now make expression from the rest of the operand
@ -586,7 +585,8 @@ bool MipsAsmParser::parseRelocOperand(const MCExpr *&Res) {
const AsmToken &nextTok = Parser.getTok(); const AsmToken &nextTok = Parser.getTok();
if (nextTok.isNot(AsmToken::Identifier)) if (nextTok.isNot(AsmToken::Identifier))
return true; return true;
Str = StringRef(Str.str() + "(%" + nextTok.getIdentifier().str()); Str += "(%";
Str += nextTok.getIdentifier();
Parser.Lex(); //eat identifier Parser.Lex(); //eat identifier
if (getLexer().getKind() != AsmToken::LParen) if (getLexer().getKind() != AsmToken::LParen)
return true; return true;
@ -603,9 +603,9 @@ bool MipsAsmParser::parseRelocOperand(const MCExpr *&Res) {
return true; //parenthesis must follow reloc operand return true; //parenthesis must follow reloc operand
//Check the type of the expression //Check the type of the expression
if (MCConstantExpr::classof(IdVal)) { if (const MCConstantExpr *MCE = dyn_cast<MCConstantExpr>(IdVal)) {
//it's a constant, evaluate lo or hi value //it's a constant, evaluate lo or hi value
int Val = ((const MCConstantExpr*)IdVal)->getValue(); int Val = MCE->getValue();
if (Str == "lo") { if (Str == "lo") {
Val = Val & 0xffff; Val = Val & 0xffff;
} else if (Str == "hi") { } else if (Str == "hi") {
@ -615,9 +615,9 @@ bool MipsAsmParser::parseRelocOperand(const MCExpr *&Res) {
return false; return false;
} }
if (MCSymbolRefExpr::classof(IdVal)) { if (const MCSymbolRefExpr *MSRE = dyn_cast<MCSymbolRefExpr>(IdVal)) {
//it's a symbol, create symbolic expression from symbol //it's a symbol, create symbolic expression from symbol
StringRef Symbol = ((const MCSymbolRefExpr*)IdVal)->getSymbol().getName(); StringRef Symbol = MSRE->getSymbol().getName();
MCSymbolRefExpr::VariantKind VK = getVariantKind(Str); MCSymbolRefExpr::VariantKind VK = getVariantKind(Str);
Res = MCSymbolRefExpr::Create(Symbol,VK,getContext()); Res = MCSymbolRefExpr::Create(Symbol,VK,getContext());
return false; return false;