[PATCH] D43708: [DAE] don't remove args of musttail target/caller

[Fedor Indutny via Phabricator via llvm-commits]

indutny added inline comments.


================
Comment at: lib/Transforms/IPO/DeadArgumentElimination.cpp:512
+
+  for (Function::const_iterator BB = F.begin(), E = F.end(); BB != E; ++BB) {
+    if (const ReturnInst *RI = dyn_cast<ReturnInst>(BB->getTerminator())) {
----------------
aprantl wrote:
> range-based-for?
FWIW, I've just added the braces to the existing code. It seemed to be out-of-scope change when I was working on it. Should I do it now?


https://reviews.llvm.org/D43708