[llvm] [RISCV][GlobalISel] Legalize Scalable Vector Loads (PR #84965)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 25 15:04:18 PDT 2024


================
@@ -220,7 +220,8 @@ struct TypePairAndMemDesc {
            Align >= Other.Align &&
            // FIXME: This perhaps should be stricter, but the current legality
            // rules are written only considering the size.
-           MemTy.getSizeInBits() == Other.MemTy.getSizeInBits();
----------------
topperc wrote:

I guess since `isCompatible` just checks the size, I guess `{v16s8, p0, s128, 8}` or `{v16s8, p0, v16s8, 8}` are equivalent for AArch64?

So using `{nxv1s8, p0, nxv1s8, 8}` would be correct for RISC-V.

https://github.com/llvm/llvm-project/pull/84965


More information about the llvm-commits mailing list