[PATCH] D126563: [RISCV] Allow PRE of vsetvli involving non-1 LMUL

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 15:36:44 PDT 2022


reames added inline comments.


================
Comment at: llvm/test/CodeGen/RISCV/rvv/sink-splat-operands.ll:4181
 ; CHECK-NEXT:    # =>This Inner Loop Header: Depth=1
 ; CHECK-NEXT:    vsetvli zero, a3, e32, m8, ta, mu
 ; CHECK-NEXT:    vle32.v v8, (a0)
----------------
craig.topper wrote:
> I guess this didn't optimized because the amount was in a register?
Yep, large constant AVLs are probably a case we need to handle explicitly.  Haven't quite fully thought through what we want there.  


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126563/new/

https://reviews.llvm.org/D126563



More information about the llvm-commits mailing list