diff options
author | Connor Abbott <[email protected]> | 2015-08-03 14:38:12 -0700 |
---|---|---|
committer | Emil Velikov <[email protected]> | 2015-12-04 16:37:07 +0000 |
commit | 38c645b60a49899e58edce4a2b47ed19b25c3b08 (patch) | |
tree | fba598cb0e9fc62a5b5fbf4554eb662d56a1360d /src/glsl/glcpp/tests | |
parent | 2dff4c6fa76acf3c415ee406faa1005f301b09fb (diff) |
i965: fix 64-bit immediates in brw_inst(_set)_bits
If we tried to get/set something that was exactly 64 bits, we would
try to do (1 << 64) - 1 to calculate the mask which doesn't give us all
1's like we want.
v2 (Iago)
- Replace ~0 by ~0ull
- Removed unnecessary parenthesis
v3 (Kristian)
- Avoid the conditional
Reviewed-by: Iago Toral Quiroga <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Reviewed-by: Kristian Høgsberg <[email protected]>
(cherry picked from commit b1a83b5d1b677faf650a41cd2c152b4d1cd18f84)
Squashed with commit
i965: Use ull immediates in brw_inst_bits
This fixes a regression introduced in b1a83b5d1 that caused basically all
shaders to fail to compile on 32-bit platforms.
Reported-by: Mark Janes <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
(cherry picked from commit 9d703de85a7bdd6ae767aedd7690cb82794cb90a)
Nominated-by: Ian Romanick <[email protected]>
Diffstat (limited to 'src/glsl/glcpp/tests')
0 files changed, 0 insertions, 0 deletions