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

llvmpipe: Fix deferred depth writes for Z16_UNORM. 

This special path hadn't been exercised by my earlier testing, and mask
values weren't being properly truncated to match the values.

This change fixes that.

Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
This commit is contained in:
José Fonseca 2013-01-29 10:45:01 +00:00
parent 0eb588a37c
commit 3b683700ef
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/gallium/drivers/llvmpipe/lp_bld_depth.c
View File

@ -893,6 +893,7 @@ lp_build_deferred_depth_write(struct gallivm_state *gallivm,
struct lp_build_context z_bld;
LLVMValueRef z_dst;
LLVMBuilderRef builder = gallivm->builder;
LLVMValueRef mask_value;
/* XXX: pointlessly redo type logic:
*/
@ -904,11 +905,15 @@ lp_build_deferred_depth_write(struct gallivm_state *gallivm,
z_dst = LLVMBuildLoad(builder, zs_dst_ptr, "zsbufval");
mask_value = lp_build_mask_value(mask);
if (z_type.width < z_src_type.width) {
/* Truncate incoming ZS and mask values (e.g., when writing to Z16_UNORM) */
zs_value = LLVMBuildTrunc(builder, zs_value, z_bld.vec_type, "");
mask_value = LLVMBuildTrunc(builder, mask_value, z_bld.int_vec_type, "");
}
z_dst = lp_build_select(&z_bld, lp_build_mask_value(mask), zs_value, z_dst);
z_dst = lp_build_select(&z_bld, mask_value, zs_value, z_dst);
LLVMBuildStore(builder, z_dst, zs_dst_ptr);
}