Test legalization for (s7, s8, s16, s32, s48, s64, s96) for rv32, (s8, s15, s16, s32, s64, s72, s128, s192) for rv64.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/Target/RISCV/GISel/RISCVLegalizerInfo.cpp | ||
---|---|---|
38 | Shouldn't that be DstSize > Xlen not DstSize >= 2 * XLen? Thought it won't make a difference since anything between XLen and XLen*2 isn't a power of 2 so the !isPowerOf2_32(DstSize) will reject it. |
llvm/lib/Target/RISCV/GISel/RISCVLegalizerInfo.cpp | ||
---|---|---|
38 | I think you're right. |
Can't we just handle 2 * XLen in the next if. We're not trying to support vectors so we don't need a vector exception