diff options
author | NAKAMURA Takumi <geek4civic@gmail.com> | 2013-10-16 08:22:49 +0000 |
---|---|---|
committer | NAKAMURA Takumi <geek4civic@gmail.com> | 2013-10-16 08:22:49 +0000 |
commit | 272416fda946b1aa87c069bd4eba237498fe0ae8 (patch) | |
tree | 3b8d5b21864468ff9bfb26fedda9ffe86d61fc55 /llvm/lib/MC/MCParser/AsmParser.cpp | |
parent | edecc3839553adedd004b87081b9fa248df7a428 (diff) | |
download | bcm5719-llvm-272416fda946b1aa87c069bd4eba237498fe0ae8.tar.gz bcm5719-llvm-272416fda946b1aa87c069bd4eba237498fe0ae8.zip |
Revert r192758 (and r192759), "MC: Better handling of tricky symbol and section names"
GNU AS didn't like quotes in symbol names.
Error: junk at end of line, first unrecognized character is `"'
.def "@feat.00";
"@feat.00" = 1
Reproduced on Cygwin's 2.23.52.20130309 and mingw32's 2.20.1.20100303.
llvm-svn: 192775
Diffstat (limited to 'llvm/lib/MC/MCParser/AsmParser.cpp')
-rw-r--r-- | llvm/lib/MC/MCParser/AsmParser.cpp | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/llvm/lib/MC/MCParser/AsmParser.cpp b/llvm/lib/MC/MCParser/AsmParser.cpp index 50fe415c897..9a362563a7e 100644 --- a/llvm/lib/MC/MCParser/AsmParser.cpp +++ b/llvm/lib/MC/MCParser/AsmParser.cpp @@ -792,25 +792,19 @@ bool AsmParser::parsePrimaryExpr(const MCExpr *&Res, SMLoc &EndLoc) { EndLoc = SMLoc::getFromPointer(Identifier.end()); // This is a symbol reference. - StringRef SymbolName = Identifier; - MCSymbolRefExpr::VariantKind Variant = MCSymbolRefExpr::VK_None; std::pair<StringRef, StringRef> Split = Identifier.split('@'); - - if (Split.first.size() != Identifier.size() && - FirstTokenKind != AsmToken::String) { - SymbolName = Split.first; - StringRef VariantName = Split.second; - - // Lookup the symbol variant. - Variant = MCSymbolRefExpr::getVariantKindForName(VariantName); + MCSymbol *Sym = getContext().GetOrCreateSymbol(Split.first); + + // Lookup the symbol variant if used. + MCSymbolRefExpr::VariantKind Variant = MCSymbolRefExpr::VK_None; + if (Split.first.size() != Identifier.size()) { + Variant = MCSymbolRefExpr::getVariantKindForName(Split.second); if (Variant == MCSymbolRefExpr::VK_Invalid) { Variant = MCSymbolRefExpr::VK_None; - return TokError("invalid variant '" + VariantName + "'"); + return TokError("invalid variant '" + Split.second + "'"); } } - MCSymbol *Sym = getContext().GetOrCreateSymbol(SymbolName); - // If this is an absolute variable reference, substitute it now to preserve // semantics in the face of reassignment. if (Sym->isVariable() && isa<MCConstantExpr>(Sym->getVariableValue())) { |