diff options
| author | Ian Romanick <[email protected]> | 2019-02-27 15:53:55 -0800 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2019-03-06 08:35:45 -0800 |
| commit | 55e6454d5e9dae6f8f29992af83f99217446da38 (patch) | |
| tree | c178267dc4085a3f72c33e7a24cdc0141aa05c54 /src/compiler | |
| parent | 4aaf139ea4cc7c4703e1906e0074f87f76c8e4cc (diff) | |
intel/fs: Fix extract_u8 of an odd byte from a 64-bit integer
In the old code, we would generate the exact same instruction for
extract_u8(some_u64, 0) and extract_u8(some_u64, 1). The mask-a-word
trick only works for even numbered bytes.
This fixes the (new) piglit test
tests/spec/arb_gpu_shader_int64/execution/fs-ushr-and-mask.shader_test.
v2: Use a SHR instead of an AND. This saves an instruction compared to
using two moves. Suggested by Jason.
Fixes: 6ac2d169019 ("i965/fs: Fix extract_i8/u8 to a 64-bit destination")
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/compiler')
0 files changed, 0 insertions, 0 deletions