From 2756b629171f61ca8e162be7b332e91a62c5c978 Mon Sep 17 00:00:00 2001 From: Vasily Khoruzhick Date: Tue, 10 Mar 2020 01:30:14 -0700 Subject: [PATCH] lima/gpir: add better lowering for ftrunc GP doesn't support ftrunc natively and unfortunately one in generic opt_algebraic is not GP-friendly either. Introduce our own lowering that utilizes fsign() that GP supports: ftrunc(a) = fmul(fsign(a), ffloor(fmax(a, -a))) Tested-by: Andreas Baierl Reviewed-by: Andreas Baierl Signed-off-by: Vasily Khoruzhick Part-of: --- .gitlab-ci/deqp-lima-fails.txt | 97 ------------------- src/gallium/drivers/lima/ir/lima_ir.h | 1 + .../drivers/lima/ir/lima_nir_algebraic.py | 7 ++ src/gallium/drivers/lima/lima_program.c | 12 +-- 4 files changed, 11 insertions(+), 106 deletions(-) diff --git a/.gitlab-ci/deqp-lima-fails.txt b/.gitlab-ci/deqp-lima-fails.txt index c6d1cf56ecd..28948b2adf0 100644 --- a/.gitlab-ci/deqp-lima-fails.txt +++ b/.gitlab-ci/deqp-lima-fails.txt @@ -56,106 +56,9 @@ dEQP-GLES2.functional.fragment_ops.depth_stencil.random.7 dEQP-GLES2.functional.fragment_ops.depth_stencil.random.8 dEQP-GLES2.functional.fragment_ops.depth_stencil.random.9 dEQP-GLES2.functional.fragment_ops.depth_stencil.write_mask.stencil -dEQP-GLES2.functional.shaders.algorithm.hsl_to_rgb_vertex -dEQP-GLES2.functional.shaders.functions.array_arguments.global_in_int_vertex -dEQP-GLES2.functional.shaders.functions.array_arguments.local_in_int_vertex -dEQP-GLES2.functional.shaders.functions.datatypes.int_int_vertex -dEQP-GLES2.functional.shaders.functions.overloading.builtin_sin_vertex -dEQP-GLES2.functional.shaders.functions.overloading.builtin_step_vertex -dEQP-GLES2.functional.shaders.functions.overloading.user_func_arg_int_types_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.inout_highp_int_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.inout_int_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.inout_lowp_int_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.out_highp_int_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.out_int_vertex -dEQP-GLES2.functional.shaders.functions.qualifiers.out_lowp_int_vertex dEQP-GLES2.functional.shaders.loops.do_while_dynamic_iterations.vector_counter_fragment dEQP-GLES2.functional.shaders.loops.for_dynamic_iterations.vector_counter_fragment dEQP-GLES2.functional.shaders.loops.while_dynamic_iterations.vector_counter_fragment -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.highp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.lowp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_effect.mediump_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.highp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.lowp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub_assign_result.mediump_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_int_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_int_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_int_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.highp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_int_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_int_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_int_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.lowp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_int_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_int_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_int_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec2_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec3_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec4_int_vertex -dEQP-GLES2.functional.shaders.operator.binary_operator.sub.mediump_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.highp_int_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.highp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.highp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.highp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.lowp_int_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.lowp_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.lowp_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.lowp_ivec4_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.mediump_int_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.mediump_ivec2_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.mediump_ivec3_vertex -dEQP-GLES2.functional.shaders.operator.unary_operator.minus.mediump_ivec4_vertex dEQP-GLES2.functional.shaders.random.all_features.fragment.37 dEQP-GLES2.functional.shaders.random.exponential.fragment.11 dEQP-GLES2.functional.shaders.random.exponential.fragment.12 diff --git a/src/gallium/drivers/lima/ir/lima_ir.h b/src/gallium/drivers/lima/ir/lima_ir.h index b160dc96738..8bd72c55abe 100644 --- a/src/gallium/drivers/lima/ir/lima_ir.h +++ b/src/gallium/drivers/lima/ir/lima_ir.h @@ -65,6 +65,7 @@ struct ra_regs *ppir_regalloc_init(void *mem_ctx); void lima_nir_lower_uniform_to_scalar(nir_shader *shader); bool lima_nir_scale_trig(nir_shader *shader); +bool lima_nir_lower_ftrunc(nir_shader *shader); bool lima_nir_split_load_input(nir_shader *shader); #endif diff --git a/src/gallium/drivers/lima/ir/lima_nir_algebraic.py b/src/gallium/drivers/lima/ir/lima_nir_algebraic.py index 86f769587aa..db0ccc75bf5 100644 --- a/src/gallium/drivers/lima/ir/lima_nir_algebraic.py +++ b/src/gallium/drivers/lima/ir/lima_nir_algebraic.py @@ -32,6 +32,11 @@ scale_trig = [ (('fcos', 'a'), ('fcos', ('fmul', 'a', 1.0 / (2.0 * pi)))), ] +# GP has fsign op, so we can use cheaper lowering than one in generic opt_algebraic +lower_ftrunc = [ + (('ftrunc', 'a'), ('fmul', ('fsign', 'a'), ('ffloor', ('fmax', 'a', ('fneg', 'a'))))) +] + def main(): parser = argparse.ArgumentParser() parser.add_argument('-p', '--import-path', required=True) @@ -47,6 +52,8 @@ def run(): print(nir_algebraic.AlgebraicPass("lima_nir_scale_trig", scale_trig).render()) + print(nir_algebraic.AlgebraicPass("lima_nir_lower_ftrunc", + lower_ftrunc).render()) if __name__ == '__main__': main() diff --git a/src/gallium/drivers/lima/lima_program.c b/src/gallium/drivers/lima/lima_program.c index 6440a8826fc..33a695557a6 100644 --- a/src/gallium/drivers/lima/lima_program.c +++ b/src/gallium/drivers/lima/lima_program.c @@ -49,7 +49,6 @@ static const nir_shader_compiler_options vs_nir_options = { .lower_sub = true, .lower_flrp32 = true, .lower_flrp64 = true, - .lower_ftrunc = true, /* could be implemented by clamp */ .lower_fsat = true, .lower_bitops = true, @@ -123,6 +122,7 @@ lima_program_optimize_vs_nir(struct nir_shader *s) NIR_PASS(progress, s, nir_opt_cse); NIR_PASS(progress, s, nir_opt_peephole_select, 8, true, true); NIR_PASS(progress, s, nir_opt_algebraic); + NIR_PASS(progress, s, lima_nir_lower_ftrunc); NIR_PASS(progress, s, nir_opt_constant_folding); NIR_PASS(progress, s, nir_opt_undef); NIR_PASS(progress, s, nir_opt_loop_unroll, @@ -132,14 +132,8 @@ lima_program_optimize_vs_nir(struct nir_shader *s) } while (progress); NIR_PASS_V(s, nir_lower_int_to_float); - /* Run opt_algebraic between int_to_float and bool_to_float because - * int_to_float emits ftrunc, and ftrunc lowering generates bool ops - */ - do { - progress = false; - NIR_PASS(progress, s, nir_opt_algebraic); - } while (progress); - + /* int_to_float pass generates ftrunc, so lower it */ + NIR_PASS(progress, s, lima_nir_lower_ftrunc); NIR_PASS_V(s, nir_lower_bool_to_float); NIR_PASS_V(s, nir_copy_prop); -- 2.30.2