[PATCH] D134400: [RISCV] Improve support for vector fp_to_sint_sat/uint_sat.

Han-Kuan Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 21 20:53:32 PDT 2022


HanKuanChen added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:1919-1923
+    DstContainerVT = getContainerForFixedLengthVector(DAG, DstVT, Subtarget);
+    SrcContainerVT = getContainerForFixedLengthVector(DAG, SrcVT, Subtarget);
+    assert(DstContainerVT.getVectorElementCount() ==
+               SrcContainerVT.getVectorElementCount() &&
+           "Expected same element count");
----------------
check `DstVT.getVectorNumElements()` and `SrcVT.getVectorNumElements()` before `getContainerForFixedLengthVector`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134400



More information about the llvm-commits mailing list