[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