[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