From bd50f9947905feb5199c7cfe29c640084f882199 Mon Sep 17 00:00:00 2001
From: Yong He <yonghe@outlook.com>
Date: Wed, 4 Dec 2024 21:19:25 -0800
Subject: Make fvk-invert-y work on mesh shader ouptuts. (#5760)

Co-authored-by: Ellie Hermaszewska <ellieh@nvidia.com>
---
 source/slang/slang-ir-vk-invert-y.cpp | 67 ++++++++++++++++++++---------------
 1 file changed, 38 insertions(+), 29 deletions(-)

(limited to 'source/slang')

diff --git a/source/slang/slang-ir-vk-invert-y.cpp b/source/slang/slang-ir-vk-invert-y.cpp
index 8a59472dd..e7fc81144 100644
--- a/source/slang/slang-ir-vk-invert-y.cpp
+++ b/source/slang/slang-ir-vk-invert-y.cpp
@@ -1,6 +1,7 @@
 #include "slang-ir-vk-invert-y.h"
 
 #include "slang-ir-insts.h"
+#include "slang-ir-util.h"
 #include "slang-ir.h"
 
 namespace Slang
@@ -28,37 +29,45 @@ void invertYOfPositionOutput(IRModule* module)
         {
             // Find all loads and stores to it.
             IRBuilder builder(module);
-            traverseUses(
-                globalInst,
-                [&](IRUse* use)
+            List<IRUse*> useWorkList;
+            auto processUse = [&](IRUse* use)
+            {
+                if (auto store = as<IRStore>(use->getUser()))
                 {
-                    if (auto store = as<IRStore>(use->getUser()))
-                    {
-                        if (store->getPtr() != globalInst)
-                            return;
+                    if (getRootAddr(store->getPtr()) != globalInst)
+                        return;
 
-                        builder.setInsertBefore(store);
-                        auto originalVal = store->getVal();
-                        auto invertedVal = _invertYOfVector(builder, originalVal);
-                        builder.replaceOperand(&store->val, invertedVal);
-                    }
-                    else if (auto load = as<IRLoad>(use->getUser()))
-                    {
-                        // Since we negate the y coordinate before writing
-                        // to gl_Position, we also need to negate the value after reading from it.
-                        builder.setInsertAfter(load);
-                        // Store existing uses of the load that we are going to replace with
-                        // inverted val later.
-                        List<IRUse*> oldUses;
-                        for (auto loadUse = load->firstUse; loadUse; loadUse = loadUse->nextUse)
-                            oldUses.add(loadUse);
-                        // Get the inverted vector.
-                        auto invertedVal = _invertYOfVector(builder, load);
-                        // Replace original uses with the invertex vector.
-                        for (auto loadUse : oldUses)
-                            builder.replaceOperand(loadUse, invertedVal);
-                    }
-                });
+                    builder.setInsertBefore(store);
+                    auto originalVal = store->getVal();
+                    auto invertedVal = _invertYOfVector(builder, originalVal);
+                    builder.replaceOperand(&store->val, invertedVal);
+                }
+                else if (auto load = as<IRLoad>(use->getUser()))
+                {
+                    // Since we negate the y coordinate before writing
+                    // to gl_Position, we also need to negate the value after reading from it.
+                    builder.setInsertAfter(load);
+                    // Store existing uses of the load that we are going to replace with
+                    // inverted val later.
+                    List<IRUse*> oldUses;
+                    for (auto loadUse = load->firstUse; loadUse; loadUse = loadUse->nextUse)
+                        oldUses.add(loadUse);
+                    // Get the inverted vector.
+                    auto invertedVal = _invertYOfVector(builder, load);
+                    // Replace original uses with the invertex vector.
+                    for (auto loadUse : oldUses)
+                        builder.replaceOperand(loadUse, invertedVal);
+                }
+                else if (auto getElementPtr = as<IRGetElementPtr>(use->getUser()))
+                {
+                    traverseUses(getElementPtr, [&](IRUse* use) { useWorkList.add(use); });
+                }
+            };
+            traverseUses(globalInst, processUse);
+            for (Index i = 0; i < useWorkList.getCount(); i++)
+            {
+                processUse(useWorkList[i]);
+            }
         }
     }
 }
-- 
cgit v1.2.3