Hi Yuao, thanks for the patch. — I have one nit:
Yuao Ma wrote:
This patch cleans up a duplicate test driver. Regression-tested. OK for trunk?
...
Subject: [PATCH] fortran: cleanup duplicate tests for c_f_pointer_shape_driver; nfc
'; nfc'?
tests_2_driver and tests_4_driver are identical, and tests_4_driver is not used at all. This patch clean this up, and format the driver with gcc style.
...
* gfortran.dg/c_f_pointer_shape_tests_2_driver.c: Removed. * gfortran.dg/c_f_pointer_shape_tests_4_driver.c: Removed. * gfortran.dg/c_f_pointer_shape_tests_driver.c: New test.
This sounds as if c_f_pointer_shape_tests_driver.c is new. But it is just the renamed _2_driver plus some formatting. (As can be gathered from the wording above the ChangeLog). I have bluntly updated the ChangeLog entry to * gfortran.dg/c_f_pointer_shape_tests_2_driver.c: Renamed to ... * gfortran.dg/c_f_pointer_shape_tests_driver.c: ... this; format with gcc style. and pushed it as r16-3031-gde2eaf63ad4334 / https://gcc.gnu.org/r16-3031-gde2eaf63ad4334 Thanks for the cleanup! Tobias