[PATCH] D105168: [RISCV] Unify the arch string parsing logic to RISCVISAInfo.

Yueh-Ting Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 25 01:57:53 PDT 2021


eopXD added inline comments.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:268
+// omitted from the version string. E.g., rv32i2p0, rv32i2, rv32i2p1.
+static Error getExtensionVersion(StringRef MArch, StringRef Ext, StringRef In,
+                                 unsigned &Major, unsigned &Minor,
----------------
For naming consistency with `parseArchString`, maybe rename `MArch` to `Arch`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105168/new/

https://reviews.llvm.org/D105168



More information about the cfe-commits mailing list