From 1a4a2f563b04e93cca68d0b7013e52d949b1613b Mon Sep 17 00:00:00 2001 From: Jordan Justen Date: Tue, 3 Mar 2020 02:28:24 -0800 Subject: [PATCH] intel/compiler/cs: Allow simd32 in some more cases with no8 and/or no16 If no16 was specified, and the shader can't run in simd8 due to the local_size, then we need to generate a simd32 program. If both no8 and no16 are specified, then we need to generate a simd32 program. Rework: * Drop update of `if` that would have changed `do32` to try simd32 even if simd16 spilled registers. (Caio) Signed-off-by: Jordan Justen Reviewed-by: Caio Marcelo de Oliveira Filho Part-of: --- src/intel/compiler/brw_fs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index bab816227c0..e10c6705858 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -9054,7 +9054,7 @@ brw_compile_cs(const struct brw_compiler *compiler, void *log_data, * * TODO: Use performance_analysis and drop this boolean. */ - const bool needs_32 = min_dispatch_width > 16 || + const bool needs_32 = v == NULL || (INTEL_DEBUG & DEBUG_DO32) || generate_all; -- 2.30.2