[ET Trac] [Einstein Toolkit] #1382: Cactus should disable collapse sections automatically for C _and_ Fortran

Einstein Toolkit trac-noreply at einsteintoolkit.org
Tue Jun 11 21:51:53 CDT 2013


#1382: Cactus should disable collapse sections automatically for C _and_ Fortran
--------------------------+-------------------------------------------------
 Reporter:  knarf         |       Owner:            
     Type:  defect        |      Status:  new       
 Priority:  minor         |   Milestone:            
Component:  Cactus        |     Version:  ET_2013_05
 Keywords:  openmp flesh  |  
--------------------------+-------------------------------------------------
 Currently, Cactus disables OpenMP collapse due to compiler problems either
 when specified explicitly in the option list, or when finding certain
 compilers that are known to have this kind of problem. However, the latter
 is only done for C and C++, but not Fortran. In addition, Cactus aborts
 compilation if collapse is not explicitly disabled, but affected compilers
 are found, with a message that collapse is disabled for C, but not for
 Fortran, indicating an error in the option list. While the first is true
 since Cactus actually disabled the collapse sections because it found a
 faulty compiler, the latter is not, since the option list doesn't mention
 collapse sections at all and isn't inconsistent.

 This means in the end that the auto-detections isn't really working in the
 way that is was intended: that Cactus does things automatically if faulty
 compilers are found. Instead, Cactus currently aborts, with a message that
 isn't leading users in the right direction. A workaround right now is to
 define CCTK_DISABLE_OMP_COLLAPSE for both C and Fortran in the option
 list.

 The proposed patch removes this error message, and instead adds the same
 compiler-detection for Fortran, that is already present for C and C++. The
 advantage would be that option lists wouldn't need to define anything.
 Cactus would either disable the collapse sections for all languages, or
 for none.

 If accepted, I propose to backport this change, in addition to applying it
 to the development version. The patch was created using ET_2013_05.

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1382>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit


More information about the Trac mailing list