Add VADD/VADS/VADX/VSUB/VSBS/VSBX/VMPY/VMPS/VMPX/VMPD/VDIV/VDVS/VDVX
instructions. Also add regression tests.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/Target/VE/VEInstrVec.td | ||
---|---|---|
507 | Shouldn't the RC register class for pv.*.up instructions be F32? |
Add reason and idea.
llvm/lib/Target/VE/VEInstrVec.td | ||
---|---|---|
507 | I remember why I use I64 here. This instruction is vector integer add instruction. I didn't use F32 here since our F32 has represented only f32 values. It doesn't represent i32 values. However, it may be possible to assign not only f32 but also i32 to F32 register class and use F32 here. Similarly, it is also possible to assign both f32 and i32 to I32 register class too since VE has vector floating-point add instruction like "PVFADDLO". Let me consider this. |
llvm/lib/Target/VE/VEInstrVec.td | ||
---|---|---|
507 | I like your idea of mapping f32/i32 to both I32 and F32. AFAIK, the mapping between types (eg i32) and registers (eg F32) only matters for isel pattern constraints (custom lowering code is not subject to those constraints). |
Change my mind. Stop previous plan to change.
llvm/lib/Target/VE/VEInstrVec.td | ||
---|---|---|
507 | I like that idea too. But, I understand doing that needs a lot of work. Can we merge this one as is and try to optimize it later? For example, it is possible to define f32 and i32 to both F32 and I32. However, it doesn't help ISel patterns at the moment. Let consider about implementing AND for higher i32 and lower i32. What kind of patterns we can to write? Pat<(i32 (and ... doesn't work. Maybe we can write Pat<(I32 (and... and Pat<(F32 (and.... I've not tried it yet. Then, we also need a lot of patterns to make it worth. So, let me merge all vector instructions first. |
It looks like all of these modifications will be overwritten by internal modifications, so I'm abandoning all of vector modifications once.
Shouldn't the RC register class for pv.*.up instructions be F32?