[PATCH] D52018: [AMDGPU] Add instruction selection for i1 to f16 conversion

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 14 08:34:22 PDT 2018


critson added a comment.

In https://reviews.llvm.org/D52018#1232860, @nhaehnle wrote:

> Why the detour via V_CVT_F16_F32 instead of selecting an fp16 1.0 constant directly?


It is not possible select an fp16 constant with V_CNDMASK_B32.
In principle the VOP2 version could be used to select an inline literal, but only when the i1 is vcc.


Repository:
  rL LLVM

https://reviews.llvm.org/D52018





More information about the llvm-commits mailing list