Skip to content

Fix openmp#42

Open
addman2 wants to merge 2 commits intosissaschool:develfrom
addman2:fix-openmp
Open

Fix openmp#42
addman2 wants to merge 2 commits intosissaschool:develfrom
addman2:fix-openmp

Conversation

@addman2
Copy link
Collaborator

@addman2 addman2 commented May 13, 2023

Fix problem with OpenMP. CPU only.

@addman2 addman2 changed the base branch from main to devel May 13, 2023 16:02
@addman2
Copy link
Collaborator Author

addman2 commented May 13, 2023

This is strange that it failed, I need to investigate this.

@kousuke-nakano
Copy link
Collaborator

kousuke-nakano commented May 13, 2023 via email

@kousuke-nakano
Copy link
Collaborator

Dear @addman2, did you pinpoint the problem?

@kousuke-nakano
Copy link
Collaborator

Dear @addman2 , I will investigate this problem if you have not solved it. Let me know :-)

@addman2
Copy link
Collaborator Author

addman2 commented Jun 17, 2024

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.

@kousuke-nakano
Copy link
Collaborator

Dear @addman2, thanks!!! Let me know if there is something I can contribute to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants