commit e6d803fd4a383cecf8c643095f093a31c944b785 Author: Robert-André Mauchin Date: Wed Apr 3 01:36:52 2019 +0200 Fix for GCC9 new OpenMP data sharing GCC 9 started implementing the OpenMP 4.0 and later behavior. When not using default clause or when using default(shared), this makes no difference, but if using default(none), previously the choice was not specify the const qualified variables on the construct at all, or specify in firstprivate clause. In GCC 9 as well as for OpenMP 4.0 compliance, those variables need to be specified on constructs in which they are used, either in shared or in firstprivate clause. Specifying them in firstprivate clause is one way to achieve compatibility with both older GCC versions and GCC 9, another option is to drop the default(none) clause. This patch thus drops the default(none) clause. See https://gcc.gnu.org/gcc-9/porting_to.html#ompdatasharing Signed-off-by: Robert-André Mauchin diff --git a/intern/elbeem/intern/solver_main.cpp b/intern/elbeem/intern/solver_main.cpp index 68f7c04cd54..514087b6130 100644 --- a/intern/elbeem/intern/solver_main.cpp +++ b/intern/elbeem/intern/solver_main.cpp @@ -381,7 +381,7 @@ LbmFsgrSolver::mainLoop(const int lev) GRID_REGION_INIT(); #if PARALLEL==1 const int gDebugLevel = ::gDebugLevel; -#pragma omp parallel default(none) num_threads(mNumOMPThreads) \ +#pragma omp parallel num_threads(mNumOMPThreads) \ reduction(+: \ calcCurrentMass,calcCurrentVolume, \ calcCellsFilled,calcCellsEmptied, \ @@ -1126,7 +1126,7 @@ LbmFsgrSolver::preinitGrids() GRID_REGION_INIT(); #if PARALLEL==1 const int gDebugLevel = ::gDebugLevel; -#pragma omp parallel default(none) num_threads(mNumOMPThreads) \ +#pragma omp parallel num_threads(mNumOMPThreads) \ reduction(+: \ calcCurrentMass,calcCurrentVolume, \ calcCellsFilled,calcCellsEmptied, \ @@ -1164,7 +1164,7 @@ LbmFsgrSolver::standingFluidPreinit() GRID_REGION_INIT(); #if PARALLEL==1 const int gDebugLevel = ::gDebugLevel; -#pragma omp parallel default(none) num_threads(mNumOMPThreads) \ +#pragma omp parallel num_threads(mNumOMPThreads) \ reduction(+: \ calcCurrentMass,calcCurrentVolume, \ calcCellsFilled,calcCellsEmptied, \ ='left'>AgeCommit message (Expand)Author 2020-07-25utils: Move <location> and '&error-location' to (guix diagnostics)....Ludovic Courtès 2020-02-09Update e-mail address for Jakob L. Kreuze....Tobias Geerinckx-Rice 2019-12-08machine: Remove unnecessary record self-referencing bindings....Ludovic Courtès 2019-12-07machine: Add provenance tracking to each machine operating system....Ludovic Courtès 2019-08-15machine: Implement 'roll-back-machine'....Jakob L. Kreuze 2019-08-06machine: Rename 'system' field....Jakob L. Kreuze 2019-07-06gnu: Add machine type for deployment specifications....Jakob L. Kreuze