[Beignet] [PATCH] GBE: tweak register expire frequency on simd16 mode.

Zhigang Gong zhigang.gong at intel.com
Mon Jun 16 21:56:31 PDT 2014


According to Yongjia's test report, it's better to keep
the same freqency of expiration with both simd8 and simd16
mode.

Signed-off-by: Zhigang Gong <zhigang.gong at intel.com>
---
 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 d225bdb..02e0cde 100644
--- a/backend/src/backend/gen_reg_allocation.cpp
+++ b/backend/src/backend/gen_reg_allocation.cpp
@@ -950,8 +950,7 @@ namespace gbe
     // The tradeoff here is to reduce the freqency here. And if we are under spilling
     // then no need to reduce that freqency as the register pressure is the most
     // important factor.
-    if (tick % (ctx.getSimdWidth() == 8 ? 12 : 4) == 0
-        || ctx.reservedSpillRegs != 0)
+    if (tick % 12 == 0 || ctx.reservedSpillRegs != 0)
       this->expireGRF(interval);
     tick++;
     // For some scalar byte register, it may be used as a destination register
-- 
1.8.3.2



More information about the Beignet mailing list