#2344: presync fails to parse WaveToyOpenCL's schedule.ccl
| Reporter: | Roland Haas |
| Status: | new |
| Milestone: | |
| Version: | |
| Type: | bug |
| Priority: | critical |
| Component: | EinsteinToolkit thorn |
Comment (by Roland Haas):
SYNC is definitely not case sensitive. In the same schedule.ccl file there is:
SYNC: Scalar
which compiles and works fine.
Note that the reason why schedule groups are case sensitive is very different from the one why scheduled routines written in C are. The latter are case sensitive since the linker symbol that is the C function preserves the case that is used in the C source code (which is not the case for Fortran scheduled function which are not case sensitive). Thus you get a link time error if you provide the wrong case for a C scheduled routine.ON the other hand, schedule group names never show up as a C identifier anywhere and are case sensitive because the flesh’s C code (in CCTKi_DoScheduleFunction) uses Util_GetHandle to get the group handle, which function uses strcmp on the handle name, which is the scheduled group name as it appears in schedule.ccl.
If there was a problem making READ / WRITE case insenstitive then that problem should be fixed.One problem that comes to mind would be generating variable names in DECLARE_CCTK_ARGUMENTS_Checked if one used the READ / WRITE strings in a case sensitive comparison against the full list that is currently used.
--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2344/presync-fails-to-parse-wavetoyopencls