From: Francisco Jerez Date: Wed, 18 May 2016 06:52:15 +0000 (-0700) Subject: i965/fs: Add missing get_latency_gen7() cases for the Gen7 pull constant opcodes. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e531b7907a6a10922e09c42f9c78d3b59beab2b4;p=mesa.git i965/fs: Add missing get_latency_gen7() cases for the Gen7 pull constant opcodes. This was causing the scheduler to be rather optimistic about the latency of pull constant opcodes on Gen7+. This might seem to increase the cycle count estimate calculated by the scheduler itself for some shaders, even though the actual cycle count should actually be decreased. Reviewed-by: Jason Ekstrand --- diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp index 4011e0f3538..6d6a19d0288 100644 --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp +++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp @@ -297,7 +297,9 @@ schedule_node::set_latency_gen7(bool is_haswell) break; case FS_OPCODE_VARYING_PULL_CONSTANT_LOAD_GEN4: + case FS_OPCODE_VARYING_PULL_CONSTANT_LOAD_GEN7: case FS_OPCODE_UNIFORM_PULL_CONSTANT_LOAD: + case FS_OPCODE_UNIFORM_PULL_CONSTANT_LOAD_GEN7: case VS_OPCODE_PULL_CONSTANT_LOAD: /* testing using varying-index pull constants: *