Started by an SCM change12912407142b14155687d0d80b174796965d2793b1212407142b14155687d0d80b174796965d2793b12origin/master12407142b14155687d0d80b174796965d2793b1212407142b14155687d0d80b174796965d2793b12origin/master113b19b67713baf175f99baa0cb0bdde6e8c1fbc588b19b67713baf175f99baa0cb0bdde6e8c1fbc588origin/grid_fixesb19b67713baf175f99baa0cb0bdde6e8c1fbc588b19b67713baf175f99baa0cb0bdde6e8c1fbc588origin/grid_fixes108b928767c7238139d599dc55420d2a0956161726ab928767c7238139d599dc55420d2a0956161726aorigin/feeding_gr_to_hostb928767c7238139d599dc55420d2a0956161726ab928767c7238139d599dc55420d2a0956161726aorigin/feeding_gr_to_host5012090eb2e167c2bd651d84762128d49894a7d54602090eb2e167c2bd651d84762128d49894a7d5460refs/remotes/origin/master2090eb2e167c2bd651d84762128d49894a7d54602090eb2e167c2bd651d84762128d49894a7d5460refs/remotes/origin/master2090eb2e167c2bd651d84762128d49894a7d54602090eb2e167c2bd651d84762128d49894a7d5460refs/remotes/origin/masterhttps://github.com/larson-group/clubb.git5013e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d143e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d14refs/remotes/origin/master3e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d143e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d14refs/remotes/origin/master3e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d143e2fe67aa2761b7dbf7b49d582bbd94ff2ec1d14refs/remotes/origin/masterhttps://github.com/larson-group/sys_adminfalse#5014132751912456clubb_fire_tuning_gfortran_test #501501false5011335SUCCESS1714639026433http://carson.math.uwm.edu/jenkins/job/clubb_fire_tuning_gfortran_test/501/src/CLUBB_core/advance_clubb_core_module.F90src/CLUBB_core/advance_xm_wpxp_module.F90src/CLUBB_core/advance_wp2_wp3_module.F90src/CLUBB_core/clubb_api_module.F90src/Morrison_microphys/microphysics.F90src/CLUBB_core/fill_holes.F90src/CLUBB_core/advance_xp2_xpyp_module.F903abb325943da1a557e1213b2908ead3c828aff161714587276000http://carson.math.uwm.edu/jenkins/user/hueblerGunther Hueblerhuebler@uwm.eduMaking num_draw_points in fill_holes a constant. We were already using this value as a constant everywhere, but passing a constant by argument list makes it difficult/impossible for a compiler to optimize using that constant, unless it does inlining. Now, rather than passing the constant num_hf_draw_points (or sometimes a hardcoded 2) we just use num_hf_draw_points directly from constants_clubb. This massively improves the performance of a loop in fill holes when using nvhpc+omp, which was the motivation for this, but should also improve the performance on CPUs. Everything is BFB. 2024-05-01 13:14:36 -05003abb325943da1a557e1213b2908ead3c828aff16Making num_draw_points in fill_holes a constant. We were already using this value as a constant everywhere, but passing a constant by argument list makes it difficult/impossible for a compiler to optimize using that constant, unless it does inlining. Now, rather than passing the constant num_hf_draw_points (or sometimes a hardcoded 2) we just use num_hf_draw_points directly from constants_clubb. This massively improves the performance of a loop in fill holes when using nvhpc+omp, which was the motivation for this, but should also improve the performance on CPUs. Everything is BFB.editsrc/CLUBB_core/clubb_api_module.F90editsrc/Morrison_microphys/microphysics.F90editsrc/CLUBB_core/advance_clubb_core_module.F90editsrc/CLUBB_core/fill_holes.F90editsrc/CLUBB_core/advance_xm_wpxp_module.F90editsrc/CLUBB_core/advance_xp2_xpyp_module.F90editsrc/CLUBB_core/advance_wp2_wp3_module.F90src/Morrison_microphys/microphysics.F902090eb2e167c2bd651d84762128d49894a7d54601714622486000http://carson.math.uwm.edu/jenkins/user/pickettCole Pickettlarsongroupgithub@gmail.comRaw-copy updating src/Morrison_microphys 2024-05-01 23:01:26 -05002090eb2e167c2bd651d84762128d49894a7d5460Raw-copy updating src/Morrison_microphyseditsrc/Morrison_microphys/microphysics.F90githttp://carson.math.uwm.edu/jenkins/user/pickettCole Picketthttp://carson.math.uwm.edu/jenkins/user/hueblerGunther Hueblerfalse502http://carson.math.uwm.edu/jenkins/job/clubb_fire_tuning_gfortran_test/502/500http://carson.math.uwm.edu/jenkins/job/clubb_fire_tuning_gfortran_test/500/