diff options
author | Matt Turner <[email protected]> | 2017-01-20 13:35:33 -0800 |
---|---|---|
committer | Francisco Jerez <[email protected]> | 2017-04-14 14:56:09 -0700 |
commit | 2eeb1b0ad9453ba135b72aaeec6c0d4dbf9ac87c (patch) | |
tree | 095ff6975dad93898fc6ca54c2d0ed151a0b3750 /src/intel/Android.genxml.mk | |
parent | d8441e2276912d353d4fc6c0cf6b781ab5153ee7 (diff) |
i965: Use correct VertStride on align16 instructions.
In commit c35fa7a, we changed the "width" of DF source registers to 2,
which is conceptually fine. Unfortunately a VertStride of 2 is not
allowed by align16 instructions on IVB/BYT, and the regular VertStride
of 4 works fine in any case.
See generated_tests/spec/arb_gpu_shader_fp64/execution/built-in-functions/vs-round-double.shader_test
for example:
cmp.ge.f0(8) g18<1>DF g1<0>.xyxyDF -g8<2>DF { align16 1Q };
ERROR: In Align16 mode, only VertStride of 0 or 4 is allowed
cmp.ge.f0(8) g19<1>DF g1<0>.xyxyDF -g9<2>DF { align16 2N };
ERROR: In Align16 mode, only VertStride of 0 or 4 is allowed
v2:
- Add spec quote (Curro).
- Change the condition to only BRW_VERTICAL_STRIDE_2 (Curro)
Reviewed-by: Samuel Iglesias Gonsálvez <[email protected]>
Reviewed-by: Francisco Jerez <[email protected]>
Diffstat (limited to 'src/intel/Android.genxml.mk')
0 files changed, 0 insertions, 0 deletions