RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2025-02-01 05:01:59 +01:00
Code Issues Projects Releases Wiki Activity

[PowerPC, AsmParser] Enable the mnemonic spell corrector

Browse Source 
r307148 added an assembly mnemonic spelling correction support and enabled it
on ARM. This enables that support on PowerPC as well.

Patch by Dmitry Venikov, thanks!

Differential Revision: https://reviews.llvm.org/D40552

llvm-svn: 320911
This commit is contained in:
Hal Finkel 2017-12-16 02:42:18 +00:00
parent ab792ca03d
commit 7b5dc23d75
2 changed files with 59 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/Target/PowerPC/AsmParser/PPCAsmParser.cpp
View File

@ -393,6 +393,10 @@ public:
/// getEndLoc - Get the location of the last token of this operand.
SMLoc getEndLoc() const override { return EndLoc; }
/// getLocRange - Get the range between the first and last token of this
/// operand.
SMRange getLocRange() const { return SMRange(StartLoc, EndLoc); }
/// isPPC64 - True if this operand is for an instruction in 64-bit mode.
bool isPPC64() const { return IsPPC64; }
@ -1268,6 +1272,9 @@ void PPCAsmParser::ProcessInstruction(MCInst &Inst,
}
}
static std::string PPCMnemonicSpellCheck(StringRef S, uint64_t FBS,
unsigned VariantID = 0);
bool PPCAsmParser::MatchAndEmitInstruction(SMLoc IDLoc, unsigned &Opcode,
OperandVector &Operands,
MCStreamer &Out, uint64_t &ErrorInfo,
@ -1283,8 +1290,13 @@ bool PPCAsmParser::MatchAndEmitInstruction(SMLoc IDLoc, unsigned &Opcode,
return false;
case Match_MissingFeature:
return Error(IDLoc, "instruction use requires an option to be enabled");
case Match_MnemonicFail:
return Error(IDLoc, "unrecognized instruction mnemonic");
case Match_MnemonicFail: {
uint64_t FBS = ComputeAvailableFeatures(getSTI().getFeatureBits());
std::string Suggestion = PPCMnemonicSpellCheck(
((PPCOperand &)*Operands[0]).getToken(), FBS);
return Error(IDLoc, "invalid instruction" + Suggestion,
((PPCOperand &)*Operands[0]).getLocRange());
}
case Match_InvalidOperand: {
SMLoc ErrorLoc = IDLoc;
if (ErrorInfo != ~0ULL) {
@ -1920,6 +1932,7 @@ extern "C" void LLVMInitializePowerPCAsmParser() {
#define GET_REGISTER_MATCHER
#define GET_MATCHER_IMPLEMENTATION
#define GET_MNEMONIC_SPELL_CHECKER
#include "PPCGenAsmMatcher.inc"
// Define this matcher function after the auto-generated include so we

44
test/MC/PowerPC/invalid-instructions-spellcheck.s Normal file
View File

@ -0,0 +1,44 @@
# RUN: not llvm-mc -triple powerpc-unknown-unknown < %s 2>&1 | FileCheck %s
# This tests the mnemonic spell checker.
# First check what happens when an instruction is omitted:
%r1, %r2, %r3
# CHECK: error: unexpected token at start of statement
# CHECK-NEXT: %r1, %r2, %r3
# CHECK-NEXT: ^
# We don't want to see a suggestion here; the edit distance is too large to
# give sensible suggestions:
aaaaaaaaaaaaaaa %r1, %r2, %r3
# CHECK: error: invalid instruction
# CHECK-NEXT: aaaaaaaaaaaaaaa %r1, %r2, %r3
# CHECK-NEXT: ^
# Check that we get one suggestion: 'vmaxfpg' is 1 edit away, i.e. an deletion.
vmaxfpg %r1, %r2
# CHECK: error: invalid instruction, did you mean: vmaxfp?
# CHECK-NEXT: vmaxfpg %r1, %r2
# CHECK-NEXT: ^
# Check edit distance 1 and 2, just insertions:
xsnmsubad %r1, %r2
# CHECK: error: invalid instruction, did you mean: xsmsubadp, xsnmsubadp?
# CHECK-NEXT: xsnmsubad %r1, %r2
# CHECK-NEXT: ^
# Check an instruction that is 2 edits away, and also has a lot of candidates:
adXd %r1, %r2, %r3
# CHECK: error: invalid instruction, did you mean: add, addc, adde, addi, fadd?
# CHECK-NEXT: adXd %r1, %r2, %r3
# CHECK-NEXT: ^