LGTM, thanks.
-----Original Message-----
From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of
xionghu....@intel.com
Sent: Monday, March 21, 2016 9:11 PM
To: beignet@lists.freedesktop.org
Cc: Luo, Xionghu <xionghu....@intel.com>
Subject: [Beignet] [PATCH] [OCL20] atomic bug fix.
From: Luo Xionghu <xionghu....@intel.com>
should use the dst type to differentiate the long type atomic, the src type
will always be 64 bit as it is the address register.
Signed-off-by: Luo Xionghu <xionghu....@intel.com>
---
backend/src/backend/gen8_encoder.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/backend/src/backend/gen8_encoder.cpp
b/backend/src/backend/gen8_encoder.cpp
index 9af8cee..b83979c 100644
--- a/backend/src/backend/gen8_encoder.cpp
+++ b/backend/src/backend/gen8_encoder.cpp
@@ -206,7 +206,7 @@ namespace gbe
this->setDst(insn, GenRegister::uw16grf(dst.nr, 0));
this->setSrc0(insn, GenRegister::ud8grf(src.nr, 0));
this->setSrc1(insn, GenRegister::immud(0));
- int type_long = (src.type == GEN_TYPE_UL || src.type == GEN_TYPE_L) ? 1: 0;
+ int type_long = (dst.type == GEN_TYPE_UL || dst.type == GEN_TYPE_L)
+ ? 1: 0;
setAtomicA64MessageDesc(insn, function, bti.value.ud, srcNum, type_long);
}
--
2.1.4
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet
_______________________________________________
Beignet mailing list
Beignet@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/beignet
