[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