diff options
author | Luke Lau <luke@igalia.com> | 2024-10-10 21:40:24 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-10-10 14:40:24 +0100 |
commit | a3cd269fbebecb6971e216a9c29ad8933ad7b0fc (patch) | |
tree | c7a9b003718301b9801bc59c236b38a38e8e58c9 /llvm/lib/CodeGen/MachineCopyPropagation.cpp | |
parent | 6779376ee917279b16e256839d236cfdf8fd9ab9 (diff) | |
download | llvm-a3cd269fbebecb6971e216a9c29ad8933ad7b0fc.zip llvm-a3cd269fbebecb6971e216a9c29ad8933ad7b0fc.tar.gz llvm-a3cd269fbebecb6971e216a9c29ad8933ad7b0fc.tar.bz2 |
[RISCV] Remove {s,u}int_to_fp custom op action for f16/bf16 (#111471)
It turns out that {s,u}int_to_fp nodes get their operation action from
their operand's type, not the result type, so we don't need to set it
for fp16 or bf16. vp_{s,u}int_to_fp uses the result type though so we
need to keep it.
This also means that we can lower int_to_fp for fixed length bf16
vectors already, so this adds tests for that.
The cost model test changes are due to BasicTTIImpl's getCastInstrCost
not taking into account that int_to_fp needs its legal type swapped.
This can be fixed in a later patch, but its worth noting that the
affected types in the tests currently crash when lowered anyway (due to
them needing split at LMUL > 8)
Diffstat (limited to 'llvm/lib/CodeGen/MachineCopyPropagation.cpp')
0 files changed, 0 insertions, 0 deletions