KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

i965/fs_surface_builder: Mask signed integers after conversion

This commit is contained in:
Jason Ekstrand 2016-01-18 17:30:59 -08:00
parent ba393c9d81
commit cc065e0ad7
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp
View File

@ -717,6 +717,15 @@ namespace {
bld.emit_minmax(offset(dst, bld, c), offset(dst, bld, c),
brw_imm_d(-(int)scale(widths[c] - s) - 1),
BRW_CONDITIONAL_GE);
/* Mask off all but the bits we actually want. Otherwise, if
* we pass a negative number into the hardware when it's
* expecting something like UINT8, it will happily clamp it to
* +255 for us.
*/
if (is_signed && widths[c] < 32)
bld.AND(offset(dst, bld, c), offset(dst, bld, c),
brw_imm_d((1 << widths[c]) - 1));
}
}
@ -787,6 +796,15 @@ namespace {
/* Convert to integer. */
bld.RNDE(offset(fdst, bld, c), offset(fdst, bld, c));
bld.MOV(offset(dst, bld, c), offset(fdst, bld, c));
/* Mask off all but the bits we actually want. Otherwise, if
* we pass a negative number into the hardware when it's
* expecting something like UINT8, it will happily clamp it to
* +255 for us.
*/
if (is_signed && widths[c] < 32)
bld.AND(offset(dst, bld, c), offset(dst, bld, c),
brw_imm_d((1 << widths[c]) - 1));
}
}