diff options
author | Kenneth Graunke <[email protected]> | 2015-01-17 01:01:35 -0800 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2015-01-17 14:20:41 -0800 |
commit | 461103ef64858e9d81073fad1bd8222b70b2754b (patch) | |
tree | 4eb084bb579cbc74bbf54e9eda8e5c80c7e61060 /src/glsl/tests/builtin_variable_test.cpp | |
parent | 8c6018e9bcdd3067d35fa942f0c5b7e229d837d0 (diff) |
i965/fs: Fix the dummy fragment shader.
We hit an assertion that the destination of the FB write should not be
an immediate. (I don't know what we were thinking.) Use ARF null.
Trying to substitute real shaders with the dummy shader would crash
when trying to upload non-existent uniforms. Say there are none.
It also wouldn't generate any code because we didn't compute the CFG,
and code generation now requires it. Compute it.
Gen4-5 also require a message header to be present.
On Gen6+, there were assertion failures in SF/SBE state because
urb_setup was memset to 0 instad of -1, causing it to think there were
attributes when nothing was set up right. Set to no attributes.
Finally, you have to ensure "Setup URB Entry Read Length" is non-zero
or you get GPU hangs, at least on Crestline.
It now works on at least Crestline and Haswell.
Signed-off-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/glsl/tests/builtin_variable_test.cpp')
0 files changed, 0 insertions, 0 deletions