Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| llvm/lib/Target/RISCV/RISCVISelLowering.cpp | ||
|---|---|---|
| 428 | 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 | ||
|---|---|---|
| 6589 | 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 | ||
|---|---|---|
| 432 | 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?