[PATCH] D78849: [MachineVerifier] retrofit iterators with range for. NFC

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 13:28:02 PDT 2020


nickdesaulniers added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:178
+        return llvm::any_of(
+            RM, [this](const auto &P) { return addRequired(P.first); });
       }
----------------
build failures look like:

```
llvm/llvm/lib/CodeGen/MachineVerifier.cpp:178:70: error: cannot call member function ‘bool {anonymous}::MachineVerifier::BBInfo::addRequired(unsigned int)’ without object

error: void value not ignored as it ought to be
```

Not sure why I don't see these locally?  Does the lambda need `this->addRequired`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78849





More information about the llvm-commits mailing list