radv/aco: Don't lower subtractions
authorDaniel Schürmann <daniel@schuermann.dev>
Wed, 25 Sep 2019 14:34:29 +0000 (16:34 +0200)
committerDaniel Schürmann <daniel@schuermann.dev>
Mon, 30 Sep 2019 09:44:10 +0000 (09:44 +0000)
40228 shaders in 20236 tests
Totals:
SGPRS: 2045512 -> 2046496 (0.05 %)
VGPRS: 1430856 -> 1430464 (-0.03 %)
Spilled SGPRs: 1077 -> 1077 (0.00 %)
Spilled VGPRs: 0 -> 0 (0.00 %)
Private memory VGPRs: 0 -> 0 (0.00 %)
Scratch size: 10348 -> 10348 (0.00 %) dwords per thread
Code Size: 77202840 -> 77151832 (-0.07 %) bytes
LDS: 863 -> 863 (0.00 %) blocks
Max Waves: 260729 -> 260754 (0.01 %)

Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
src/amd/vulkan/radv_shader.c

index 086f7cd86167eaa7f2ec807a2d8fad40e2278ace..5fe361d44cf79e0dacfc135af298dfef0335d498 100644 (file)
@@ -92,7 +92,6 @@ static const struct nir_shader_compiler_options nir_options_aco = {
        .lower_fdiv = true,
        .lower_bitfield_insert_to_bitfield_select = true,
        .lower_bitfield_extract = true,
-       .lower_sub = true, /* TODO: set this to false once !1236 is merged */
        .lower_pack_snorm_2x16 = true,
        .lower_pack_snorm_4x8 = true,
        .lower_pack_unorm_2x16 = true,