Details
Diff Detail
Event Timeline
llvm/lib/Target/RISCV/RISCVISelLowering.cpp | ||
---|---|---|
476 | Why do we need to Custom legalize the narrow scalar types? Shouldn't they be promoted by generic type legalization PromoteIntOp? |
llvm/lib/Target/RISCV/RISCVISelLowering.cpp | ||
---|---|---|
6307 | Should we be using INTRINSIC_W_CHAIN and intrinsic_riscv_vlse/vlse_mask similar to what we do for other VP load/store? |
Changelog:
- Remove unnecessary custom legalization of narrow scalar types
- Do not add new RISCVISD nodes, use existing intrinsics instead
llvm/test/CodeGen/RISCV/rvv/strided-vpload.ll | ||
---|---|---|
146 | Use poison instead of undef |
llvm/test/CodeGen/RISCV/rvv/fixed-vectors-strided-vpload.ll | ||
---|---|---|
26 | This should have ".i32" on the end of the intrinsic name. Same for all of the intrinsics with i32 stride. |
llvm/lib/Target/RISCV/RISCVISelLowering.cpp | ||
---|---|---|
478 | Why do we need custom legalization for i32 on rv64? Shouldn't we have a generic PromoteIntOp for this? |
Why do we need to Custom legalize the narrow scalar types? Shouldn't they be promoted by generic type legalization PromoteIntOp?