[all-commits] [llvm/llvm-project] 115672: [RISCV] Remove now redundant isElementRotate shuff...

Philip Reames via All-commits all-commits at lists.llvm.org
Fri Feb 21 09:18:18 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 115672f57edb05717684126ada86ff89e3ec0ee8
      https://github.com/llvm/llvm-project/commit/115672f57edb05717684126ada86ff89e3ec0ee8
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2025-02-21 (Fri, 21 Feb 2025)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp

  Log Message:
  -----------
  [RISCV] Remove now redundant isElementRotate shuffle lowering [NFCI] (#128064)

This is the first cleanup following the introduction of the new
isMaskedSlidePair lowering in 43f2968. As mentioned in that review, the
new code is a generalization of the existing isElementRotate, but was
staged to make diffs manageable.

This change removes isElementRotate. The tricky bit is making sure that
a) the same shuffles hit the same lowering paths (in particular, two
element shuffles can match multiple ways) and 2) avoiding DAG
canonicalization changes.



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