[all-commits] [llvm/llvm-project] f4e906: [RISCV] Move more stuff to RISCVInstrInfoZvfbf.td....

Craig Topper via All-commits all-commits at lists.llvm.org
Thu Oct 16 21:26:56 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f4e906a7b6d39f0206f1f6c27f67c88bd747f44d
      https://github.com/llvm/llvm-project/commit/f4e906a7b6d39f0206f1f6c27f67c88bd747f44d
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2025-10-16 (Thu, 16 Oct 2025)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVFeatures.td
    M llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td
    M llvm/lib/Target/RISCV/RISCVInstrInfoZvfbf.td

  Log Message:
  -----------
  [RISCV] Move more stuff to RISCVInstrInfoZvfbf.td. Fix incorrect Predicates. (#163846)

The PseudoVFWCVTBF16_F_F and PseudoVFNCVTBF16_F_F pseudos should only
have Zvfbfmin as their Predicate, they are not used by Zvfofp8min which
requires different Pseudos with different SEW.

The patterns had an outer Predicates saying ZvfbfminOrZvfofp8min and
inner Predicate saying HasVInstructionsMinimal||HasVInstructionsF32. I
believe the outer Predicates overrides the inner Predicates.

The correct predicate for the patterns is Zvfbfmin(which implies
HasVInstructionsF32). They aren't used by Zvfofp8min and
HasVInstructionsMinimal is going to be changed to Zvbfmin||Zvfbfa, but
these instructions are Zvfbfmin only.



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


More information about the All-commits mailing list