-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix openmp #42
base: devel
Are you sure you want to change the base?
Fix openmp #42
Conversation
This is strange that it failed, I need to investigate this. |
Dear Oto,
Merge pull request #29 from kousuke-nakano/devel-#24 <https://github.com/sissaschool/turborvb/actions/runs/4967748101>
Yes, this is strange. The zgemm failed :-(
… On May 14, 2023, at 1:21, addman2 ***@***.***> wrote:
This is strange that it failed, I need to investigate this.
—
Reply to this email directly, view it on GitHub <#42 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AI7IYQNHHBT75UT4QDVM4SDXF6YITANCNFSM6AAAAAAYASKRCI>.
You are receiving this because your review was requested.
|
Dear @addman2, did you pinpoint the problem? |
Dear @addman2 , I will investigate this problem if you have not solved it. Let me know :-) |
Hi Kosuke, it was partly solved. When turbo is compiled with the old build system (actually new build system based on makefile it was completly redesigned) the problem disappeared. I wrote the makefile build system to solve this issue. So now we know the problem is somewhere in CMake. I need to do careful comparision between those two to find where is the problem. Towards end of this week I will be more free so I can look at it again. |
Dear @addman2, thanks!!! Let me know if there is something I can contribute to. |
Fix problem with OpenMP. CPU only.