From e473fbe4690b5cbe3769042a4917f22559e2ba8d Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Wed, 10 Dec 2014 14:56:46 -0800 Subject: vc4: Add support for turning constant uniforms into small immediates. Small immediates have the downside of taking over the raddr B field, so you might have less chance to pack instructions together thanks to raddr B conflicts. However, it also reduces some register pressure since it lets you load 2 "uniform" values in one instruction (avoiding a previous load of the constant value to a register), and increases some pairing for the same reason. total uniforms in shared programs: 16231 -> 13374 (-17.60%) uniforms in affected programs: 10280 -> 7423 (-27.79%) total instructions in shared programs: 40795 -> 41168 (0.91%) instructions in affected programs: 25551 -> 25924 (1.46%) In a previous version of this patch I had a reduction in instruction count by forcing the other args alongside a SMALL_IMM to be in the A file or accumulators, but that increases register pressure and had a bug in handling FRAG_Z. In this patch is I just use raddr conflict resolution, which is more expensive. I think I'd rather tweak allocation to have some way to slightly prefer good choices for files in general, rather than risk failing to register allocate by forcing things into register classes. --- src/gallium/drivers/vc4/Makefile.sources | 1 + 1 file changed, 1 insertion(+) (limited to 'src/gallium/drivers/vc4/Makefile.sources') diff --git a/src/gallium/drivers/vc4/Makefile.sources b/src/gallium/drivers/vc4/Makefile.sources index 6bcb731d034..1f8e8c41bf4 100644 --- a/src/gallium/drivers/vc4/Makefile.sources +++ b/src/gallium/drivers/vc4/Makefile.sources @@ -15,6 +15,7 @@ C_SOURCES := \ vc4_opt_copy_propagation.c \ vc4_opt_cse.c \ vc4_opt_dead_code.c \ + vc4_opt_small_immediates.c \ vc4_packet.h \ vc4_program.c \ vc4_qir.c \ -- cgit v1.2.3