[all-commits] [llvm/llvm-project] 18dd68: [RISC-V][RVY] Fix round-trip and order of arch str...

[Alexander Richardson via All-commits]

  Branch: refs/heads/users/arichardson/risc-v-rvy-rank-y-after-i-and-e-in-canonical-extension-order
  Home:   https://github.com/llvm/llvm-project
  Commit: 18dd6895afc58b5dd4829fe298e6d1ed4c91d117
      https://github.com/llvm/llvm-project/commit/18dd6895afc58b5dd4829fe298e6d1ed4c91d117
  Author: Alex Richardson <alexrichardson at google.com>
  Date:   2026-09-30 (Wed, 30 Sep 2026)

  Changed paths:
    M clang/test/Driver/print-supported-extensions-riscv.c
    M llvm/lib/Support/RISCVISAUtils.cpp
    M llvm/test/MC/RISCV/rvy-build-attributes.s
    M llvm/unittests/TargetParser/RISCVISAInfoTest.cpp

  Log Message:
  -----------
  [RISC-V][RVY] Fix round-trip and order of arch strings with 'y'

Because 'y' is not in AllStdExts, singleLetterExtensionRank previously
fell through to the unknown single-letter extension case which results
in incorrect behaviour for the RVY base ISA.
For normalized RVY ISA strings and '.attribute arch' round-tripping, we
have two options: omit 'i' when 'y' is present, or always emit both the
'i'/'e' base and 'y' (and accept 'y' immediately after 'i' or 'e' in
parseArchString). This commit does the latter to align with the "longer
base name" proposal (https://lists.riscv.org/g/tech-unprivileged/topic/longer_base_name_proposal/116854896)
of using `RV{32,64}L{I,E}Y`, Adding support for parsing 'i'/'e' also
makes it easier to support the CHERIoT 'rv32ey.

This commit was created with the help of AI tools


  Commit: 04c75f9e492e5c1601b8a00785dc353dac1ebeba
      https://github.com/llvm/llvm-project/commit/04c75f9e492e5c1601b8a00785dc353dac1ebeba
  Author: Alex Richardson <alexrichardson at google.com>
  Date:   2026-10-01 (Thu, 01 Oct 2026)

  Changed paths:
    M llvm/lib/TargetParser/RISCVISAInfo.cpp
    M llvm/test/MC/RISCV/rvy-build-attributes.s
    M llvm/unittests/TargetParser/RISCVISAInfoTest.cpp

  Log Message:
  -----------
  fix round-tripping and simpify the parsing to also allow rv32iy (matching long base name proposal)


Compare: https://github.com/llvm/llvm-project/compare/dee8f11517fa...04c75f9e492e

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications