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

Jiahan Xie via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 25 11:23:27 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();
----------------
jiahanxie353 wrote:




> Comparing the min value is losing the scalable bit. Why a fixed size and scalable size should not be compatible right?

Should scalable `nxv2s8` be compatible with `s8`? What about `nxv2s8` and `s8`? I'm a bit unsure about the logic behind

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


More information about the llvm-commits mailing list