[llvm] [RISCV] Combine vslidedown_vl with known VL to a smaller LMUL (PR #66267)
Luke Lau via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 18 02:36:55 PDT 2023
lukel97 wrote:
> This patch isn't sound. Just because the VL is known doesn't mean the slidedown offset is <= VL. The elements that will be written to the lower elements of the destination may come from the high LMUL part of the input.
Sorry you're right, I think this should be `getSmallestVTForIndex(OrigVT, VL + Offset, DL, DAG, Subtarget)`
https://github.com/llvm/llvm-project/pull/66267
More information about the llvm-commits
mailing list