Conversation
|
To clarify, I have tested this by compiling + linking a program requiring openmp |
sbc100
left a comment
There was a problem hiding this comment.
I assume the contents of system/lib/libomp is coming from an upstream repo?
Can you using git submodule for this and put it in third_party/ maybe?
Yess, it is from llvm. Since all the other llvm libraries are in this location, it made sense to put it here. Do you want the entire LLVM repository as a submodule? It seems a bit overkill, at that point the other libraries can also be pulled from there |
Oh i see, I didn't realize this was from llvm. I that case the import approach your took does seem reasonable. |
|
Failing tests:
For the last two I would like to disable the tests, but I don't know how. Ruff issues can be easily fixed ofc |
This makes
emccrecognize-fopenmp, and links it as needed.When updating openmp, it neesd to be compiled. This is not strictly neccesary, but we need to invoke cmake in order to generate the headers. When compiling openemp, we need to make use of cmake as well, as the build system is quite long and complicated. I am sure it can be done another way using ninja, and I encourage others to implement this, but I do not have the capacity right now to make this happen.
Fixes: #13892
Fixes: #17637