[ms-inline asm] Emit an error for unsupported SIZE and LENGTH directives.

Part of rdar://12576868

llvm-svn: 166792
This commit is contained in:
Chad Rosier 2012-10-26 18:32:44 +00:00
parent 0473d5429c
commit 5859356d80
1 changed files with 11 additions and 5 deletions

View File

@ -891,7 +891,7 @@ X86Operand *X86AsmParser::ParseIntelOffsetOfOperator(SMLoc Start) {
SMLoc End; SMLoc End;
const MCExpr *Val; const MCExpr *Val;
if (getParser().ParseExpression(Val, End)) if (getParser().ParseExpression(Val, End))
return 0; return ErrorOperand(Start, "Unable to parse expression!");
End = Parser.getTok().getLoc(); End = Parser.getTok().getLoc();
@ -928,7 +928,7 @@ X86Operand *X86AsmParser::ParseIntelTypeOperator(SMLoc Start) {
// identifier. // identifier.
// FIXME: Pass a valid SMLoc. // FIXME: Pass a valid SMLoc.
if (!SemaCallback->LookupInlineAsmIdentifier(Sym.getName(), NULL, Size)) if (!SemaCallback->LookupInlineAsmIdentifier(Sym.getName(), NULL, Size))
return ErrorOperand(Start, "Unable to lookup TYPE of expr."); return ErrorOperand(Start, "Unable to lookup TYPE of expr!");
Size /= 8; // Size is in terms of bits, but we want bytes in the context. Size /= 8; // Size is in terms of bits, but we want bytes in the context.
} }
@ -946,16 +946,22 @@ X86Operand *X86AsmParser::ParseIntelOperand() {
SMLoc Start = Parser.getTok().getLoc(), End; SMLoc Start = Parser.getTok().getLoc(), End;
// offset operator. // offset operator.
const AsmToken &Tok = Parser.getTok(); StringRef AsmTokStr = Parser.getTok().getString();
if ((Tok.getString() == "offset" || Tok.getString() == "OFFSET") && if ((AsmTokStr == "offset" || AsmTokStr == "OFFSET") &&
isParsingInlineAsm()) isParsingInlineAsm())
return ParseIntelOffsetOfOperator(Start); return ParseIntelOffsetOfOperator(Start);
// Type directive. // Type directive.
if ((Tok.getString() == "type" || Tok.getString() == "TYPE") && if ((AsmTokStr == "type" || AsmTokStr == "TYPE") &&
isParsingInlineAsm()) isParsingInlineAsm())
return ParseIntelTypeOperator(Start); return ParseIntelTypeOperator(Start);
// Unsupported directives.
if (isParsingIntelSyntax() &&
(AsmTokStr == "size" || AsmTokStr == "SIZE" ||
AsmTokStr == "length" || AsmTokStr == "LENGTH"))
return ErrorOperand(Start, "Unsupported directive!");
// immediate. // immediate.
if (getLexer().is(AsmToken::Integer) || getLexer().is(AsmToken::Real) || if (getLexer().is(AsmToken::Integer) || getLexer().is(AsmToken::Real) ||
getLexer().is(AsmToken::Minus)) { getLexer().is(AsmToken::Minus)) {