6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-04-05 17:39:16 +00:00
Code Issues Projects Releases Wiki Activity

AsmParser: improve diagnostics for invalid variants

Browse Source 
An emitted diagnostic for an invalid relocation variant would place the caret on
the token following the relocation variant indicator or at the end of the line
if there was no following token.  This change corrects the placement of the
caret to point to the token.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@200159 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Saleem Abdulrasool 2014-01-26 22:29:43 +00:00
parent 425eb00894
commit 10096880c8
3 changed files with 26 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/MC/MCParser/AsmParser.cpp
View File

@ -836,7 +836,8 @@ bool AsmParser::parsePrimaryExpr(const MCExpr *&Res, SMLoc &EndLoc) {
Variant = MCSymbolRefExpr::VK_None;
} else {
Variant = MCSymbolRefExpr::VK_None;
return TokError("invalid variant '" + Split.second + "'");
return Error(SMLoc::getFromPointer(Split.second.begin()),
"invalid variant '" + Split.second + "'");
}
}

11
test/MC/AsmParser/variant-diagnostics-2.s Normal file
View File

@ -0,0 +1,11 @@
# RUN: not llvm-mc -triple i386-linux-gnu -filetype asm -o /dev/null 2>&1 %s \
# RUN: | FileCheck %s
.text
function:
call external@invalid
# CHECK: error: invalid variant 'invalid'
# CHECK: call external@invalid
# CHECK: ^

13
test/MC/AsmParser/variant-diagnostics.s Normal file
View File

@ -0,0 +1,13 @@
@ RUN: not llvm-mc -triple armv7-linux-eabi -filetype asm -o /dev/null 2>&1 %s \
@ RUN: | FileCheck %s
.arch armv7
.type invalid_variant,%function
invalid_variant:
bx target(invalid)
@ CHECK: error: invalid variant 'invalid'
@ CHECK: bx target(invalid)
@ CHECK: ^