[PATCH] D134897: [AMDGPU] Fix V_CMP_CLASS_F16_t16_e64 src1 type.

Thu Sep 29 12:24:12 PDT 2022

Joe_Nash added a comment.

In D134897#3824780 <https://reviews.llvm.org/D134897#3824780>, @dp wrote:

>> For some reason, we cannot encode inline literals for VSrc_b16.
>
> This is true for inline floating-point constants only. Integer inline constants may be used without limitations, so I do not see any problems using VSrc_b16 for src1.

Before D133723 <https://reviews.llvm.org/D133723>, inline floating-point constants were allowed. Arguably they should be allowed ( https://developer.amd.com/wp-content/resources/RDNA2_Shader_ISA_November2020.pdf "Note that the S1 has a format of f16 since floating point literal
constants are interpreted as 16 bit value for this opcode"), though I don't know if they would be emitted by codegen.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134897/new/

https://reviews.llvm.org/D134897