From: Francisco Jerez Date: Tue, 17 May 2016 23:01:29 +0000 (-0700) Subject: i965/fs: Assert that IF instruction with embedded compare has legal exec_size. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=122e0315480704a7c6777b994c42448d360e6774;p=mesa.git i965/fs: Assert that IF instruction with embedded compare has legal exec_size. We shouldn't encounter these right now but if we did it wouldn't be possible for the SIMD lowering pass to split it into multiple instructions because of its side effects on control flow, so just assert in order to kill the program. Reviewed-by: Jason Ekstrand --- diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index 2b6cf221f4c..df72fda49c2 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -4788,6 +4788,10 @@ get_lowered_simd_width(const struct brw_device_info *devinfo, return MIN2(devinfo->is_haswell ? 8 : ~0u, get_fpu_lowered_simd_width(devinfo, inst)); + case BRW_OPCODE_IF: + assert(inst->src[0].file == BAD_FILE || inst->exec_size <= 16); + return inst->exec_size; + case SHADER_OPCODE_RCP: case SHADER_OPCODE_RSQ: case SHADER_OPCODE_SQRT: