Re: [PATCH] fortran: add optional lower arg to c_f_pointer

2025-08-06 Thread Yuao Ma
Oops, looks like I mess up a testcase name. Fixed in this version. On 8/6/2025 10:57 PM, Yuao Ma wrote: Hi Tobias, On 8/6/2025 4:32 PM, Tobias Burnus wrote: Hi Yuao, thanks for your patch. I have two nits: * There is no diagnostic for -std=f2018 (or lower);    can you add the 'gfc_notify_std

Re: [PATCH] fortran: add optional lower arg to c_f_pointer

2025-08-06 Thread Yuao Ma
Hi Tobias, On 8/6/2025 4:32 PM, Tobias Burnus wrote: Hi Yuao, thanks for your patch. I have two nits: * There is no diagnostic for -std=f2018 (or lower);   can you add the 'gfc_notify_std (GFC_STD_F2023' ? Done. Testcase added. * I have a minor documentation nit; current wording is   at

Re: [PATCH] fortran: cleanup duplicate tests for c_f_pointer_shape_driver; nfc

2025-08-06 Thread Yuao Ma
On 8/6/2025 5:20 PM, Tobias Burnus wrote: 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'? tes

Re: [PATCH] fortran: cleanup duplicate tests for c_f_pointer_shape_driver; nfc

2025-08-06 Thread Tobias Burnus
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,

Re: [PATCH,libgfortran] Fix PR121234, bogus diagnostic

2025-08-06 Thread Manfred Schwarb
Am 05.08.25 um 21:25 schrieb Jerry D: > Hello, > > The attached patch fixes the subject bug by adding checks for ';' it specific > places to avoid eating them and/or giving false errors. > Jerry, thanks a lot for tackling this! But in an ideal world, standards would define a delimiter argument f

Re: [PATCH] fortran: add optional lower arg to c_f_pointer

2025-08-06 Thread Tobias Burnus
Hi Yuao, thanks for your patch. I have two nits: * There is no diagnostic for -std=f2018 (or lower); can you add the 'gfc_notify_std (GFC_STD_F2023' ? * I have a minor documentation nit; current wording is at https://gcc.gnu.org/onlinedocs/gfortran/C_005fF_005fPOINTER.html Namely, ... Yu

[PING] [RFC PATCH 0/2] Fortran: multi-target clones Attribute Support

2025-08-06 Thread 泽邦 贺
ping At 2025-07-24 00:54:22, "ZAMBAR" wrote: > This patch is an experimental implementation of multi-target clones >clones for Fortran, allowing the use of the `TARGET_CLONES` attribute. >It is designed to work with the syntax like: > >`!GCC$ ATTRIBUTES TARGET_CLONES("default", "avx", "avx512f")