Jason Ekstrand ja...@jlekstrand.net writes:
Where are you getting a destination stride in a load_payload? The whole
point of load_payload was to remove partial writes so this seems to go
against everything it's intended for.
--Jason
Yeah, maybe... This case was triggered by some of my image_load_store
code that ends up using byte types with stride=4 for some image formats.
On Fri, Feb 6, 2015 at 9:42 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 163aa41..8da1f47 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -3074,7 +3074,7 @@ fs_visitor::lower_load_payload()
bool force_sechalf = inst-force_sechalf
!inst-force_writemask_all;
bool toggle_sechalf = inst-dst.width == 16
- type_sz(inst-dst.type) == 4
+ type_sz(inst-dst.type) * inst-dst.stride
== 4
!inst-force_writemask_all;
for (int i = 0; i inst-regs_written; ++i) {
metadata[dst_reg + i].written = true;
--
2.1.3
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
pgpsLmQeSeN85.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
