We were creating a truncate with the default for the type, but for
VP intrinsics we have a VL that we should use.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Paths
| Differential D118406
[RISCV] Preserve VL when truncating i64 gather/scatter indices on RV32. ClosedPublic Authored by craig.topper on Jan 27 2022, 1:34 PM.
Details Summary We were creating a truncate with the default for the type, but for
Diff Detail
Event TimelineHerald added subscribers: VincentWu, luke957, achieveartificialintelligence and 25 others. · View Herald TranscriptJan 27 2022, 1:34 PM This revision is now accepted and ready to land.Jan 28 2022, 3:10 AM This revision was landed with ongoing or failed builds.Jan 28 2022, 9:25 AM Closed by commit rGea05ee90596c: [RISCV] Preserve VL when truncating i64 gather/scatter indices on RV32. (authored by craig.topper). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 404055 llvm/lib/Target/RISCV/RISCVISelLowering.cpp
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-masked-gather.ll
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-masked-scatter.ll
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-vpgather.ll
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-vpscatter.ll
llvm/test/CodeGen/RISCV/rvv/vpgather-sdnode.ll
llvm/test/CodeGen/RISCV/rvv/vpscatter-sdnode.ll
|
We still forgot to use the correct VL for the scaling shift. I'll forgive the vsext since we don't have a VP intrinsic yet.