LGTM, thanks. -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Zhigang Gong Sent: Thursday, January 23, 2014 2:27 PM To: [email protected] Cc: Gong, Zhigang Subject: [Beignet] [PATCH] GBE: allow the bool registers to be expired.
After the previous's extra liveness analysis, we can allow bool registers to be expired now. Signed-off-by: Zhigang Gong <[email protected]> --- backend/src/backend/gen_reg_allocation.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/backend/src/backend/gen_reg_allocation.cpp b/backend/src/backend/gen_reg_allocation.cpp index 4e33240..fedd730 100644 --- a/backend/src/backend/gen_reg_allocation.cpp +++ b/backend/src/backend/gen_reg_allocation.cpp @@ -294,8 +294,7 @@ namespace gbe continue; } // Ignore booleans that were allocated with flags - // if (ctx.getRegisterFamily(reg) == ir::FAMILY_BOOL && !grfBooleans.contains(reg)) { - if (ctx.sel->getRegisterFamily(reg) == ir::FAMILY_BOOL) { + if (ctx.sel->getRegisterFamily(reg) == ir::FAMILY_BOOL && + !grfBooleans.contains(reg)) { this->expiringID++; continue; } -- 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
