[PATCH] [Core] Add parallel infrastructure to lld.

Michael Spencer bigcheesegs at gmail.com
Wed Apr 10 14:24:59 PDT 2013



================
Comment at: include/lld/Core/Parallel.h:76
@@ +75,3 @@
+
+/// \biref An implementation of an Executor that runs closures on a thread pool
+///   in filo order.
----------------
Shankar Kalpathi Easwaran wrote:
> biref -> brief
Fixed.

================
Comment at: include/lld/Core/Parallel.h:126
@@ +125,3 @@
+  std::atomic<bool> _stop;
+  std::stack<std::function<void()>> _workQueue;
+  std::mutex _mutex;
----------------
Shankar Kalpathi Easwaran wrote:
> do you want to call it as workStack instead ?
Sure.


http://llvm-reviews.chandlerc.com/D649



More information about the llvm-commits mailing list