Good patches. These patches look good to me.

-----Original Message-----
From: [email protected] 
[mailto:[email protected]] On Behalf Of Ruiling Song
Sent: Wednesday, November 20, 2013 1:52 PM
To: [email protected]
Cc: Song, Ruiling
Subject: [Beignet] [PATCH 1/3] GBE: Fix null register to integer type

GEN 'mach' instruction only support integer type register.

Signed-off-by: Ruiling Song <[email protected]>
---
 backend/src/backend/gen_context.cpp |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/backend/src/backend/gen_context.cpp 
b/backend/src/backend/gen_context.cpp
index 43b3bc7..bbfc85d 100644
--- a/backend/src/backend/gen_context.cpp
+++ b/backend/src/backend/gen_context.cpp
@@ -1181,8 +1181,8 @@ namespace gbe
     loadBottomHalf(d, y);
     p->push();
     p->curr.predicate = GEN_PREDICATE_NONE;
-    I32FullMult(GenRegister::null(), e, b, c);
-    I32FullMult(GenRegister::null(), f, a, d);
+    I32FullMult(GenRegister::retype(GenRegister::null(), GEN_TYPE_D), e, b, c);
+    I32FullMult(GenRegister::retype(GenRegister::null(), GEN_TYPE_D), f, a, d);
     p->ADD(e, e, f);
     I32FullMult(f, a, b, d);
     p->ADD(e, e, f);
-- 
1.7.9.5

_______________________________________________
Beignet mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/beignet
_______________________________________________
Beignet mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/beignet

Reply via email to