On 2/11/26 5:38 AM, Andre Vehreschild wrote:
Hi Tobias,
tests for image_status() w/ some/all arguments are in the tests:
image_status_1.f08 for compile time tests, and
image_status_2.f08 for runtime tests.
May be the test just has been added by a different commit. Preparing these
patches has taken some time and it is possible, that the changes were
accidently moved to an earlier/later commit. Shall we remedy this?
Regards,
Andre
I see these test in my local tree. I will check to see what, if anything needs
to be done.
Jerry
On Wed, 11 Feb 2026 14:08:17 +0100
Tobias Burnus <[email protected]> wrote:
Jerry D wrote:
Fortran: Small fixes of coarray routines handling and code gen.
gcc/fortran/ChangeLog:
* check.cc (gfc_check_image_status): Fix argument index of
team= argument for correct error message.
* trans-intrinsic.cc (conv_intrinsic_image_status): Team=
argument is optional and is a pointer to the team handle.
* trans-stmt.cc (gfc_trans_sync): Make images argument also a
dereferencable pointer. But treat errmsg as a pointer to a
char array like in all other functions.
gcc/testsuite/ChangeLog:
* gfortran.dg/coarray_sync_memory.f90: Adapt grep pattern for
msg being only &msg.
It seems as if a testcase for 'image_status(image[, team])' is missing
as only the 'sync' change shows up in the testsuite.
Can someone create a test for the two bits in it? A compile-time check
and a runtime check? - Or is this covered anywhere else?
Otherwise, the patch LGTM.
Tobias
