Convert _OPENMP to AMREX_USE_OMP (#170)

jrood-nrel · web-flow · commit 75da770ea36e · 2026-01-14T09:21:54.000-08:00
There is currently a mix of `_OPENMP` and `AMREX_USE_OMP` in this repo.
Should we convert all to `AMREX_USE_OMP`?
diff --git a/EBGodunov/hydro_ebgodunov_extrap_vel_to_faces.cpp b/EBGodunov/hydro_ebgodunov_extrap_vel_to_faces.cpp
@@ -47,7 +47,7 @@ EBGodunov::ExtrapVelToFaces ( MultiFab const& vel,
                   w_mac.setVal(hydro_covered_val););
 
     const int ncomp = AMREX_SPACEDIM;
-#ifdef _OPENMP
+#ifdef AMREX_USE_OMP
 #pragma omp parallel if (Gpu::notInLaunchRegion())
 #endif
     {
diff --git a/EBMOL/hydro_ebmol_extrap_vel_to_faces.cpp b/EBMOL/hydro_ebmol_extrap_vel_to_faces.cpp
@@ -36,7 +36,7 @@ EBMOL::ExtrapVelToFaces ( const MultiFab&  a_vel,
     auto const& ccent = fact.getCentroid();
 #endif
 
-#ifdef _OPENMP
+#ifdef AMREX_USE_OMP
 #pragma omp parallel if (Gpu::notInLaunchRegion())
 #endif
     {
diff --git a/Godunov/hydro_godunov_extrap_vel_to_faces_2D.cpp b/Godunov/hydro_godunov_extrap_vel_to_faces_2D.cpp
@@ -28,7 +28,7 @@ Godunov::ExtrapVelToFaces ( MultiFab const& a_vel,
     Box const& domain = geom.Domain();
     const Real* dx    = geom.CellSize();
     const int ncomp = AMREX_SPACEDIM;
-#ifdef _OPENMP
+#ifdef AMREX_USE_OMP
 #pragma omp parallel if (Gpu::notInLaunchRegion())
 #endif
     {
diff --git a/Godunov/hydro_godunov_extrap_vel_to_faces_3D.cpp b/Godunov/hydro_godunov_extrap_vel_to_faces_3D.cpp
@@ -31,7 +31,7 @@ Godunov::ExtrapVelToFaces ( MultiFab const& a_vel,
     const Real* dx    = geom.CellSize();
 
     const int ncomp = AMREX_SPACEDIM;
-#ifdef _OPENMP
+#ifdef AMREX_USE_OMP
 #pragma omp parallel if (Gpu::notInLaunchRegion())
 #endif
     {
diff --git a/MOL/hydro_mol_extrap_vel_to_faces.cpp b/MOL/hydro_mol_extrap_vel_to_faces.cpp
@@ -26,7 +26,7 @@ MOL::ExtrapVelToFaces ( const MultiFab&  a_vel,
 {
     BL_PROFILE("MOL::ExtrapVelToFaces");
 
-#ifdef _OPENMP
+#ifdef AMREX_USE_OMP
 #pragma omp parallel if (Gpu::notInLaunchRegion())
 #endif
     {

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ EBGodunov::ExtrapVelToFaces ( MultiFab const& vel,`
`47`	`47`	`w_mac.setVal(hydro_covered_val););`
`48`	`48`
`49`	`49`	`const int ncomp = AMREX_SPACEDIM;`
`50`		`-#ifdef _OPENMP`
	`50`	`+#ifdef AMREX_USE_OMP`
`51`	`51`	`#pragma omp parallel if (Gpu::notInLaunchRegion())`
`52`	`52`	`#endif`
`53`	`53`	`{`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ EBMOL::ExtrapVelToFaces ( const MultiFab& a_vel,`
`36`	`36`	`auto const& ccent = fact.getCentroid();`
`37`	`37`	`#endif`
`38`	`38`
`39`		`-#ifdef _OPENMP`
	`39`	`+#ifdef AMREX_USE_OMP`
`40`	`40`	`#pragma omp parallel if (Gpu::notInLaunchRegion())`
`41`	`41`	`#endif`
`42`	`42`	`{`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ Godunov::ExtrapVelToFaces ( MultiFab const& a_vel,`
`28`	`28`	`Box const& domain = geom.Domain();`
`29`	`29`	`const Real* dx = geom.CellSize();`
`30`	`30`	`const int ncomp = AMREX_SPACEDIM;`
`31`		`-#ifdef _OPENMP`
	`31`	`+#ifdef AMREX_USE_OMP`
`32`	`32`	`#pragma omp parallel if (Gpu::notInLaunchRegion())`
`33`	`33`	`#endif`
`34`	`34`	`{`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ Godunov::ExtrapVelToFaces ( MultiFab const& a_vel,`
`31`	`31`	`const Real* dx = geom.CellSize();`
`32`	`32`
`33`	`33`	`const int ncomp = AMREX_SPACEDIM;`
`34`		`-#ifdef _OPENMP`
	`34`	`+#ifdef AMREX_USE_OMP`
`35`	`35`	`#pragma omp parallel if (Gpu::notInLaunchRegion())`
`36`	`36`	`#endif`
`37`	`37`	`{`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ MOL::ExtrapVelToFaces ( const MultiFab& a_vel,`
`26`	`26`	`{`
`27`	`27`	`BL_PROFILE("MOL::ExtrapVelToFaces");`
`28`	`28`
`29`		`-#ifdef _OPENMP`
	`29`	`+#ifdef AMREX_USE_OMP`
`30`	`30`	`#pragma omp parallel if (Gpu::notInLaunchRegion())`
`31`	`31`	`#endif`
`32`	`32`	`{`