This patch permits OpenMP to build and work (with both gcc and clang) on OpenBSD. It mostly follows what was done for FreeBSD and NetBSD, except OpenBSD does not have pthread_getattr_np support, so it follows OS X in that one instance.
Details
- Reviewers
krytarowski - Group Reviewers
Restricted Project - Commits
- rG7e1ea993e0d3: Add OpenBSD support to OpenMP
rL348726: Add OpenBSD support to OpenMP
rOMP348726: Add OpenBSD support to OpenMP
Diff Detail
- Repository
- rL LLVM
Event Timeline
runtime/src/kmp_platform.h | ||
---|---|---|
103 | How about switching this check to __x86_64__ or defined(__x86_64) || defined(__x86_64__)? |
Removed amd64 check, changed x86_64 check to x86_64__
runtime/src/kmp_platform.h | ||
---|---|---|
103 | No objections to the former (or the latter, but I did the former). |
runtime/src/z_Linux_util.cpp | ||
---|---|---|
450 | If we are going to alter this comment, perhaps it's worth to make it more generic. "Supported not by all systems" |
Update a comment, at the request of krytarowski.
runtime/src/z_Linux_util.cpp | ||
---|---|---|
450 | Sure. The comment also says it's for Linux only, which isn't true either. So I made the whole thing generic. |
I'm sorry to keep poking, but I would really appreciate this getting committed to the tree.
I am willing to work to ensure that OpenMP continues to build on OpenBSD and is not a hindrance to other platforms.
Patch looks fine from NetBSD point of view. The rest (to accept and commit) I leave to OpenMP developers. I have not worked on it so far.
CMakeLists.txt | ||
---|---|---|
7 | This comment would be updated to a some more generic one. |
runtime/src/kmp_wrapper_malloc.h | ||
---|---|---|
102 | This is beyond the scope of this patch but all platforms (Windows, Linux, BSDs, MacOSX) as far as I can tell ship alloca(3) from <stdlib.h>. The only exception of a modern system that I'm aware of, where <stdlib.h> does not work for this task is Solaris, it requires <alloca.h>. |
Hello.
I would still very much like to add OpenBSD support to OpenMP. I have been using it for the past several months.
I have uploaded a new patch that rebases the OpenBSD additions to HEAD.
Thanks.
This comment would be updated to a some more generic one.