diff options
| author | Francisco Jerez <[email protected]> | 2015-08-05 16:29:30 +0300 |
|---|---|---|
| committer | Francisco Jerez <[email protected]> | 2015-08-06 14:12:12 +0300 |
| commit | 42a18ca76057621ae7d8812b29ea2245d6ff282d (patch) | |
| tree | 8041c0d70b1b21179151c4690cc41097a695deba /src/mesa | |
| parent | e77a4a9b1f66de383043df95aada40fd5a004913 (diff) | |
i965/fs: Fix fs_inst::regs_read() for sources in the ATTR file.
Otherwise it would crash on Gen8 with scalar VS. The issue can easily
be reproduced with the following patch, but I don't see any reason why
it wouldn't be possible to end up with an ATTR argument here even
without it.
CC: [email protected]
Reviewed-by: Connor Abbott <[email protected]>
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp index 284528d1f84..1d3bdb42284 100644 --- a/src/mesa/drivers/dri/i965/brw_fs.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp @@ -808,6 +808,7 @@ fs_inst::regs_read(int arg) const case IMM: return 1; case GRF: + case ATTR: case HW_REG: return DIV_ROUND_UP(components_read(arg) * src[arg].component_size(exec_size), |