diff options
| author | Francisco Jerez <[email protected]> | 2016-05-17 16:01:29 -0700 |
|---|---|---|
| committer | Francisco Jerez <[email protected]> | 2016-05-27 23:19:22 -0700 |
| commit | 122e0315480704a7c6777b994c42448d360e6774 (patch) | |
| tree | 513e6dedf72d521495214da7d1f6dc54696f9d62 | |
| parent | 98c8bef01cae5fd70dda22fd7ac0b5694c4dfb5f (diff) | |
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 <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
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: |