[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