Mesa (master): nv50/ir: VFETCH is also considered a load for MemoryOpt

Ilia Mirkin imirkin at kemper.freedesktop.org
Tue Jun 27 00:25:07 UTC 2017


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

Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat Jun 24 17:08:11 2017 -0400

nv50/ir: VFETCH is also considered a load for MemoryOpt

This has no effect since in practice this will only play for
memory-backed files, for which VFETCH will never happen.

Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>

---

 src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
index 5dac2f64f0..c6dbd4fead 100644
--- a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
+++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
@@ -2641,7 +2641,7 @@ MemoryOpt::findRecord(const Instruction *insn, bool load, bool& isAdj) const
    Record *it = load ? loads[sym->reg.file] : stores[sym->reg.file];
 
    for (; it; it = it->next) {
-      if (it->locked && insn->op != OP_LOAD)
+      if (it->locked && insn->op != OP_LOAD && insn->op != OP_VFETCH)
          continue;
       if ((it->offset >> 4) != (sym->reg.data.offset >> 4) ||
           it->rel[0] != insn->getIndirect(0, 0) ||




More information about the mesa-commit mailing list