[PATCH] D101215: [RISCV] Use fractional LMULs for fixed length types smaller than riscv-v-vector-bits-min.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 28 11:12:51 PDT 2021
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:1184
+ // We prefer to use LMUL=1 for VLEN sized types. Use fractional lmuls for
+ // narrower types, but we can't go below LMUL=64/SEW.
unsigned EltsPerBlock = RISCV::RVVBitsPerBlock / EltVT.getSizeInBits();
----------------
frasercrmck wrote:
> Maybe this comment could be corrected/clarified. I see what it's trying to convey but saying we can't go below `LMUL=64/SEW` makes it sound like we can't have fractional types at all since they'd always be at least 1.
Oops I think I was thinking in terms of the denominator when I wrote that. So it's really SEW/64.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101215/new/
https://reviews.llvm.org/D101215
More information about the llvm-commits
mailing list