[PATCH] D106518: [RISCV] Disable EEW=64 for index values when XLEN=32.

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 6 08:57:59 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:5475
 
+  if (IndexVT.getVectorElementType() == MVT::i64 && XLenVT == MVT::i32) {
+    report_fatal_error("The V extension does not support EEW=64 for index "
----------------
Can we truncate the index to nvxXi32 instead of erroring? Would that allow us to preserve more test cases?


================
Comment at: llvm/test/CodeGen/RISCV/rvv/fixed-vectors-masked-gather.ll:1033
 
-define <8 x i64> @mgather_baseidx_sext_v8i8_v8i64(i64* %base, <8 x i8> %idxs, <8 x i1> %m, <8 x i64> %passthru) {
-; RV32-LABEL: mgather_baseidx_sext_v8i8_v8i64:
----------------
Can these test cases be preserved in an rv64 only test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106518



More information about the cfe-commits mailing list