[PATCH] D54861: [SanitizerCommon] Test `CombinedAllocator::ForEachChunk()` in unit tests.

Kuba (Brecka) Mracek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 08:53:11 PST 2018


kubamracek accepted this revision.
kubamracek added a comment.
This revision is now accepted and ready to land.

LGTM, of course (more tests, yay!).

The indentation makes this hard to read though. Maybe declare the lambda as a separate variable?


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D54861





More information about the llvm-commits mailing list