Mesa (master): i965: Make gen6_resolve_implied_move a no-op for MRF sources .

Kenneth Graunke kwg at kemper.freedesktop.org
Fri Dec 2 10:50:41 UTC 2011


Module: Mesa
Branch: master
Commit: e304aa3600f865db533d273e2c1a554cb6a54f05
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e304aa3600f865db533d273e2c1a554cb6a54f05

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon Nov 14 19:49:54 2011 -0800

i965: Make gen6_resolve_implied_move a no-op for MRF sources.

Attempting to move an MRF to a MRF is not only pointless, it will fail
because MRFs are read-only, resulting in garbage in your register.

If we already set up a MRF source, there's nothing to resolve anyway.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/drivers/dri/i965/brw_eu_emit.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 60350ca..9d8c701 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
+++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c
@@ -67,6 +67,9 @@ gen6_resolve_implied_move(struct brw_compile *p,
    if (intel->gen < 6)
       return;
 
+   if (src->file == BRW_MESSAGE_REGISTER_FILE)
+      return;
+
    if (src->file != BRW_ARCHITECTURE_REGISTER_FILE || src->nr != BRW_ARF_NULL) {
       brw_push_insn_state(p);
       brw_set_mask_control(p, BRW_MASK_DISABLE);




More information about the mesa-commit mailing list