diff options
| author | Iago Toral Quiroga <[email protected]> | 2016-06-23 10:35:50 +0200 |
|---|---|---|
| committer | Samuel Iglesias Gonsálvez <[email protected]> | 2017-01-03 11:26:51 +0100 |
| commit | 506154f704dcb9185dadcd655fd6d0603916ea97 (patch) | |
| tree | 2eac13d2524494d57d2c810a1f426febb9410061 | |
| parent | b9cd3f5b493e6fe835f1a3a2216f29bc0a4a2a93 (diff) | |
i965/vec4: extend the DWORD multiply DepCtrl restriction to all gen8 platforms
v2:
- Add Broxton as Intel's internal PRMs says that it is needed (Matt).
Reviewed-by: Matt Turner <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index 5ff0d53171a..c461f26f71e 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp @@ -908,11 +908,14 @@ vec4_visitor::is_dep_ctrl_unsafe(const vec4_instruction *inst) (reg.type == BRW_REGISTER_TYPE_UD || \ reg.type == BRW_REGISTER_TYPE_D) - /* "When source or destination datatype is 64b or operation is integer DWord + /* From the Cherryview and Broadwell PRMs: + * + * "When source or destination datatype is 64b or operation is integer DWord * multiply, DepCtrl must not be used." - * May apply to future SoCs as well. + * + * SKL PRMs don't include this restriction though. */ - if (devinfo->is_cherryview) { + if (devinfo->gen == 8 || devinfo->is_broxton) { if (inst->opcode == BRW_OPCODE_MUL && IS_DWORD(inst->src[0]) && IS_DWORD(inst->src[1])) |