[PATCH] D27685: [libFuzzer] Diff 9 - 2 - Properly use unsigned for workers, jobs and NumberOfCpuCores.

Marcos Pividori via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 14:37:14 PST 2016


mpividori added a comment.

@kcc `std::thread::hardware_concurrency()` returns unsigned, so `NumberOfCpuCores()` should return unsigned. Would you agree on this?
As `NumberOfCpuCores()` returns unsigned I thought it would be more appropriate to use unsigned for jobs and workers. If you don't want that changes, I can use an explicit cast as I mentioned above.
Just let me know what you want and I will include that changes.
Thanks


Repository:
  rL LLVM

https://reviews.llvm.org/D27685





More information about the llvm-commits mailing list