[PATCH] D119453: [AArch64] Suggest b.nfrst if the user tries b.nfirst.
Peter Waller via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 14 08:39:35 PST 2022
peterwaller-arm accepted this revision.
peterwaller-arm added a comment.
LGTM with one more consistency tweak.
================
Comment at: llvm/lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:3092
+ if (!Suggestion.empty())
+ Msg += ". Did you mean '" + Suggestion + "'?";
+ return TokError(Msg);
----------------
Thanks for the patch. Sorry, I think another pass is needed here for consistency, the messages look like so (no quotes, single sentence):
https://github.com/llvm/llvm-project/blob/779bbbf27fe631154bdfaac7a443f198d4654688/llvm/utils/TableGen/AsmMatcherEmitter.cpp#L3091
```
test/MC/AArch64/invalid-instructions-spellcheck.s
28:// CHECK: error: unrecognized instruction mnemonic, did you mean: add, addp, adds, addv, fadd, madd?
35:// CHECK-NO-NEON: error: unrecognized instruction mnemonic, did you mean: add, adds, fadd, madd?
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119453/new/
https://reviews.llvm.org/D119453
More information about the llvm-commits
mailing list