[clang] [llvm] [RISC-V][RVY] Fix round-trip and order of arch strings with 'y' (PR #227865)
Alexander Richardson via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 1 07:56:21 PDT 2026
arichardson wrote:
Thanks for the detailed rationale, I agree with you. Omitting default base components for forward compatibility makes a lot of sense, and we shouldn't emit two single-letter base ISAs. That's what the long base format will be for.
I've updated the PR to omit i in toString() when y is present and accept rv32y/rv64y in parseNormalizedArchString(), leaving E+Y and long base names for a follow-up. A nice side effect of this is that it simplifies the overall diff, and allows merging the case statements again.
https://github.com/llvm/llvm-project/pull/227865
More information about the cfe-commits
mailing list