diff options
author | Topi Pohjolainen <[email protected]> | 2017-07-19 09:25:19 +0300 |
---|---|---|
committer | Topi Pohjolainen <[email protected]> | 2017-07-20 11:32:21 +0300 |
commit | 76e2f390f9863a356d1419982dec705260d67eff (patch) | |
tree | b6404712a91618fd2a2d7e28ea5e2bbe967c9d3e /src/mesa/drivers/dri/i965/intel_state.c | |
parent | 0e8b81af7b6fa44910b1c54fe45ba455da07bf43 (diff) |
i965/miptree: Use num_samples of 1 instead of 0 for single-sampled
Patch moves "assert(brw->num_samples <= 16)" from
emit_3dstate_multisample2() to upload_multisample_state(). Latter
is the only caller of the former and passes "brw->num_samples"
as argument. Therefore it is clearer to assert in the caller.
Possible bug fix in genX(emit_3dstate_multisample2) which
doesn't have a case for num_samples == 0 in the switch
statement.
It should be noted that intel_miptree_map()/unmap() now checks
additionally for "mt->surf.samples == 1" in order to support gen6
stencil which is already transitioned to ISL. This will go away in
next patch when native miptrees start to use isl_surf::samples as
well.
Reviewed-by: Jason Ekstrand <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Signed-off-by: Topi Pohjolainen <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/intel_state.c')
0 files changed, 0 insertions, 0 deletions