[all-commits] [llvm/llvm-project] 460c1b: [RISCV][InsertVSETVLI] Rewrite scalar insert forwa...

Philip Reames via All-commits all-commits at lists.llvm.org
Tue Jan 3 14:21:05 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 460c1bd3440ebf5a601f643e992e3a0ad3471167
      https://github.com/llvm/llvm-project/commit/460c1bd3440ebf5a601f643e992e3a0ad3471167
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2023-01-03 (Tue, 03 Jan 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp
    M llvm/test/CodeGen/RISCV/rvv/vsetvli-insert.ll

  Log Message:
  -----------
  [RISCV][InsertVSETVLI] Rewrite scalar insert forward rule in terms of demanded fields

This is mostly geared at consolidating logic into one form to reduce code duplication, but also has the effect of being a slight generalization. Since these operations aren't masked, we can ignore the mask policy bit when deciding on compatibility. The previous code was overly strict in checking that both policy bits matched.

Note: There's a slight difference from the reviewed version.  The reviewed version was based on a local revision which included the isCompatible change to only check AVL if VL is used.  I apparently never landed that change, and while functional, the functional change isn't visible without this one.  I chose to role the extra change into this patch.

Differential Revision: https://reviews.llvm.org/D140147




More information about the All-commits mailing list