[PATCH] D137707: Move "auto-init" instructions to the dominator of their users

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/MoveAutoInit.cpp:72-73
+    } else if (auto *M = dyn_cast<MemoryPhi>(MA)) {
+      for (unsigned i = 0, n = M->getNumIncomingValues(); i != n; ++i)
+        WorkList.push_back(M->getIncomingValue(i));
+    }
----------------
serge-sans-paille wrote:
> nickdesaulniers wrote:
> > Are you able to use a range-for here with `MemoryPhi::incoming_values()`?
> Unfortunately not, `SmallVector::append` doesn't accept `iterator_range`. I'll implement that.
I was just thinking

```
for (MemoryAccess *MA : M->incoming_values())
  Worklist.push_back(MA);
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137707/new/

https://reviews.llvm.org/D137707