search

StephanTLavavej / mingw-distro

MinGW distro build scripts.
494 stars 55 forks source link

OpenMP Support #40

Closed RoyiAvital closed 6 years ago

RoyiAvital commented 7 years ago

Hello,

I tried compiling a file using gcc and I got the following error:

fatal error: omp.h: No such file or directory

I looked for omp.h in all the files extracted from your installer and I couldn't find it. Could it be that OpenMP Support is missing from your distribution?

Thank You.

StephanTLavavej commented 7 years ago

Duplicate of #32. Related to #26. I can't enable OpenMP as long as it requires pthreads, which I don't build.

RoyiAvital commented 7 years ago

I see...

I thought I had the holy land :-).

Is there distribution like yours (Portable with bat file to create properly configured environment) with OpenMP support?

What you created is the perfect experience, but I really need OpenMP.

Thank You.

dokipen3d commented 7 years ago

After seeing the last comment on #26 could this maybe pretty please be reconsidered? :)

StephanTLavavej commented 7 years ago

Maybe, but I'll need either lots of free time, or someone telling me exactly how to alter my build scripts.

dokipen3d commented 7 years ago

ok thanks, I'm going to have a go at running the scripts now and see how it goes :)

dokipen3d commented 7 years ago

I was able to make a small modification to the build script and now I have pthreads in gcc to enable openmp! I'll make a pull request later today so we can test it and see if there are any issues.

screen shot 2017-09-04 at 9 04 45 am
dokipen3d commented 7 years ago

done https://github.com/StephanTLavavej/mingw-distro/pull/46

Now lets find if there is anything wrong with it. If anyone wants to send me some things to compile and run, I can give it a go.

RoyiAvital commented 7 years ago

@dokipen3d , This is great news.

You can try this: https://github.com/RoyiAvital/Projects/tree/master/ImageToColumns

Thank You.

dokipen3d commented 7 years ago

Well it compiles fine. Not sure if it runs though! I've tried a few openmp helloworlds just printing info about the threads and it all seems fine. I've got my own project I'll try next which uses it a lot.

screen shot 2017-09-04 at 11 12 05 am
StephanTLavavej commented 6 years ago

I believe I can enable winpthreads and OpenMP in distro 15.2, thanks to your pull request.

dokipen3d commented 6 years ago

Excellent thanks so much STL!

RoyiAvital commented 6 years ago

Guys,

Is the issue of MinGW64 with AVX code on Windows 64 relevant to this distribution as well? I encountered this lately and I can't get a good picture what's going on.

Please see my post here - https://github.com/Alexpux/MSYS2-packages/issues/1209#issuecomment-379547589.

It is about the AVX code no aligned correctly as required by Windows 64 Bit format.

StephanTLavavej commented 6 years ago

Please don’t hijack issues (even ones you’ve opened) for unrelated issues. In this case, I agree with the MSYS2 maintainers - this is an upstream GCC issue and should be reported to upstream GCC maintainers with a proper self-contained test case.

RoyiAvital commented 6 years ago

I'm sorry. Just wanted to know if you're aware of this or maybe you have solved it in your customized distribution.

I really hope someone will solve this.

StephanTLavavej commented 6 years ago

I don't know anything about this and can't fix it. Don't hope - instead, report the bug properly.