[PATCH] D78962: fix D78849 for g++ < 7.1

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 14:01:10 PDT 2020


MaskRay added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineVerifier.cpp:172
+        return llvm::any_of(
+            RS, [this](unsigned Reg) { return this->addRequired(Reg); });
       }
----------------
I think `[&](unsigned Reg) { return addRequired(Reg); }` works as well...

There might be a `this` capturing bug in older GCC.

I seem to recall older MSVC has similar bugs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78962





More information about the llvm-commits mailing list