rmaprath added a comment.
@EricWF: Ping?
https://reviews.llvm.org/D29757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rmaprath updated this revision to Diff 89664.
rmaprath added a comment.
Different take on the patch: Externalize this function only for the
externally-thread-api variant.
This is much less intrusive. Note that I haven't added the declaration of
`__libcpp_thread_hw_concurrency()` into `__threadi
EricWF requested changes to this revision.
EricWF added a comment.
This revision now requires changes to proceed.
Same comment as https://reviews.llvm.org/D29818
I really don't like this change (and https://reviews.llvm.org/D29818) because
they start to include a ton of extra headers, and becaus
rmaprath added a comment.
Ping?
https://reviews.llvm.org/D29757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rmaprath updated this revision to Diff 88189.
rmaprath retitled this revision from "Threading support: Externalize
hardware_concurrency()" to "[libcxx] Threading support: Externalize
hardware_concurrency()".
rmaprath added a comment.
Re-based on trunk.
https://reviews.llvm.org/D29757
Files:
