[PATCH] D19412: [libcxx] Refactor pthread usage - II
Ben Craig via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 28 10:06:10 PDT 2016
bcraig added a comment.
In http://reviews.llvm.org/D19412#414374, @rmaprath wrote:
> > On a bikeshed note: is `<__os_support>` the right name? Or should it be something like `<__thread>` or `<__threading_support>`?
>
>
> I went with `__os_support` in case if we'd want to group further external dependencies (like, for example, if some non-c POSIX calls are used in the upcoming `filesystem` library). I can revert back to `__threading_support` if that's preferred.
>
> Thanks.
>
> / Asiri
I don't particularly care for <__os_support> either. Either of <__thread> or <__threading_support> are fine with me.
Note that there are already other OS specific calls that libcxx has to handle. A lot of the locale code wants to use BSD style foo_l functions that are present on Linux. I prefer to keep the locale code independent of the threading code, as the two don't overlap much.
For filesystem related items, I would expect those to go under <__filesystem> or <__filesystem_support>, or something similar.
http://reviews.llvm.org/D19412
More information about the cfe-commits
mailing list