On 8/30/25 9:16 AM, Harald Anlauf wrote:
On 8/30/25 18:04, Jerry D wrote:
On 8/30/25 8:04 AM, Paul Richard Thomas wrote:
Hi All,
This patch is only a temporary fix because the chunks in trans-
array.cc are representation dependent. As a whole, the patch is so
straightforward that the ChangeLog serves as an explanation.
Regtests with FC32/x86_64 - OK for mainline?
Paul
Yes, OK and thanks for the fix.
Regards,
Jerry
Actually - and obviously - it is NOT OK. It fails here on
gfortran.dg/value_optional_1.f90
gfortran.dg/value_optional_2.f90
because of the new non-caught NULL-pointer dereference here:
diff --git a/gcc/fortran/trans-expr.cc b/gcc/fortran/trans-expr.cc
index 69952b33eaa..771d2c24fa9 100644
--- a/gcc/fortran/trans-expr.cc
+++ b/gcc/fortran/trans-expr.cc
@@ -6520,6 +6520,18 @@ conv_dummy_value (gfc_se * parmse, gfc_expr * e,
gfc_symbol * fsym,
gcc_assert (fsym && fsym->attr.value && !fsym->attr.dimension);
+ if (e->ts.type == BT_DERIVED && e->ts.u.derived->attr.pdt_type)
Note that e == NULL for a missing actual argument, so you need to
check for this case.
Thanks,
Harald
oops, reviewed and did not apply and test. my bad.
Jerry
