Revert "[AMDGPU] Select v_sat_pk_u8_i16"
authorpvanhout <pierre.vanhoutryve@amd.com>
Fri, 31 Mar 2023 10:26:27 +0000 (12:26 +0200)
committerpvanhout <pierre.vanhoutryve@amd.com>
Fri, 31 Mar 2023 10:39:49 +0000 (12:39 +0200)
commit16e1f8e970aad4c92b31b330daf7274e89cca98d
tree9136eb992907e3a4a5e82296f1c1ee81055d0b27
parentaa48da3537c57d264aac1388f8b48da53a2f8e1c
Revert "[AMDGPU] Select v_sat_pk_u8_i16"

This reverts commit 64b45db34a0cd979dae9ca3016e9da517e57b987.

Reason: the patterns are wrong which can result in a miscompilation.
However, fixing the pattern is not trivial due to how i8 values
are handled, and due to the additional type-checking performed by
D147127: trunc/smax/smin are all defined as int ops in the DAG
despite them working on vectors too.

As this is not a much-needed pattern, I prefer reverting for now
until I can find time to properly rewrite the pattern.
llvm/lib/Target/AMDGPU/AMDGPUInstructions.td
llvm/lib/Target/AMDGPU/SIInstructions.td
llvm/test/CodeGen/AMDGPU/v_sat_pk_u8_i16.ll