OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nir: Avoid generating extra ftruncs for array handling.

Browse Source 
It's quite likely that the source of the f2i32 was already an integer, in
which case we can skip the ftrunc (particularly useful on the int-to-float
class of hardware that's unlikely to just have a native trunc opcode!).

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15870>
This commit is contained in:
Emma Anholt
2022-04-11 17:27:13 -07:00
parent e4aa5f7889
commit 66a0f318fd
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
src/compiler/nir/nir_lower_int_to_float.c
View File

@@ -65,7 +65,29 @@ lower_alu_instr(nir_builder *b, nir_alu_instr *alu)
case nir_op_b2i32: alu->op = nir_op_b2f32; break;
case nir_op_i2f32: alu->op = nir_op_mov; break;
case nir_op_u2f32: alu->op = nir_op_mov; break;
case nir_op_f2i32: alu->op = nir_op_ftrunc; break;
case nir_op_f2i32: {
alu->op = nir_op_ftrunc;
/* If the source was already integer, then we did't need to truncate and
* can switch it to a mov that can be copy-propagated away.
*/
nir_alu_instr *src_alu = nir_src_as_alu_instr(alu->src[0].src);
if (src_alu) {
switch (src_alu->op) {
case nir_op_fround_even:
case nir_op_fceil:
case nir_op_ftrunc:
case nir_op_ffloor:
alu->op = nir_op_mov;
break;
default:
break;
}
}
break;
}
case nir_op_f2u32: alu->op = nir_op_ffloor; break;
case nir_op_i2b1: alu->op = nir_op_f2b1; break;