[PATCH] D60758: Add an assertion that parallelForEach doesn't nest.

Andrew Kelley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 16 01:12:23 PDT 2019


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

This will be nice to prevent bugs in the future. Should it be behind `#ifndef NDEBUG` though? Presumably if assertions are disabled then this bug would be expected to be undefined behavior and avoid the overhead of the thread local variable, is that right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60758





More information about the llvm-commits mailing list