[PATCH] D48240: Try again to implement a FIFO task queue
Zachary Turner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 15 17:15:54 PDT 2018
zturner updated this revision to Diff 151592.
zturner added a comment.
Made the test cases a little stronger. Before I was simply assigning to the local variables in the test case. Now I increment them. This way the test will fail if a task were to ever get executed more than once, for example, whereas before it would pass since it would just be re-assigning the same value.
https://reviews.llvm.org/D48240
Files:
llvm/include/llvm/Support/TaskQueue.h
llvm/lib/Support/CMakeLists.txt
llvm/lib/Support/TaskQueue.cpp
llvm/unittests/Support/CMakeLists.txt
llvm/unittests/Support/TaskQueueTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48240.151592.patch
Type: text/x-patch
Size: 13109 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180616/099568ec/attachment.bin>
More information about the llvm-commits
mailing list