[PATCH] D109215: [RISCV] Fix arch sring parsing for multi-character extensions
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 3 02:19:27 PDT 2021
jrtc27 added a comment.
RISC-V manual, §27.6:
> Standard extensions can also be named using a single “Z” followed by an alphabetical name and an optional version number.
Numbers are not in the alphabet, therefore they cannot be part of a valid name. If you start allowing numbers in the extension name then this introduces the need for all kinds of weird edge cases to remove ambiguity, but leads to huge amounts of confusion. Just don't do it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109215/new/
https://reviews.llvm.org/D109215
More information about the llvm-commits
mailing list