public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function
@ 2024-02-27 22:11 kargl at gcc dot gnu.org
  2024-02-28  2:11 ` [Bug fortran/114141] " jvdelisle at gcc dot gnu.org
                   ` (15 more replies)
  0 siblings, 16 replies; 17+ messages in thread
From: kargl at gcc dot gnu.org @ 2024-02-27 22:11 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

            Bug ID: 114141
           Summary: ASSOCIATE and complex part ref when associate target
                    is a function
           Product: gcc
           Version: 14.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
          Assignee: unassigned at gcc dot gnu.org
          Reporter: kargl at gcc dot gnu.org
  Target Milestone: ---

Created attachment 57556
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=57556&action=edit
patch

gfortran rejects the following code with an invalid error message.
The problem is that the target is not resolved to have type 
COMPLEX. The attach patch rectifies the issue.  It has been regtested
on x86_64-*-freebsd.


! { dg-do run }
program foo
   implicit none
   real y
   associate (x => log(cmplx(-1,0)))
      y = x%im
      if (int(100*y)-314 /= 0) stop 1
   end associate
end program

gfortran13 -c a.f90
a.f90:6:13:

    6 |       y = x%im
      |             1
Error: Symbol 'x' at (1) has no IMPLICIT type
troutmask:sgk[244]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
@ 2024-02-28  2:11 ` jvdelisle at gcc dot gnu.org
  2024-02-28  2:21 ` jvdelisle at gcc dot gnu.org
                   ` (14 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: jvdelisle at gcc dot gnu.org @ 2024-02-28  2:11 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

Jerry DeLisle <jvdelisle at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jvdelisle at gcc dot gnu.org

--- Comment #1 from Jerry DeLisle <jvdelisle at gcc dot gnu.org> ---
I see the fail here but I am not sure of the validity. I am going to read the
latest standard. I always thought of associate to refer associate complicated
data references to make code more readable. In this case it is referring to a
function which resolves to a constant.

Interestingly if I wrap the associate with parens one gets:

   associate (x => (log(cmplx(-1,0))))

$ gfc pr114141.f90 
pr114141.f90:6:14:

    6 |       y = x%im
      |              1
Error: The RE or IM part_ref at (1) must be applied to a COMPLEX expression

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
  2024-02-28  2:11 ` [Bug fortran/114141] " jvdelisle at gcc dot gnu.org
@ 2024-02-28  2:21 ` jvdelisle at gcc dot gnu.org
  2024-02-28 15:43 ` kargl at gcc dot gnu.org
                   ` (13 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: jvdelisle at gcc dot gnu.org @ 2024-02-28  2:21 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #2 from Jerry DeLisle <jvdelisle at gcc dot gnu.org> ---
It looks like the 'selector' in this case is an expr.

The expr must be a pointer object or a 'designator'

A designator must be:

R901
designator

object-name
array-element
array-section
coindexed-named-object
complex-part-designator
structure-component
substring

I am not seeing the expr in the example as one of these listed. ???

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
  2024-02-28  2:11 ` [Bug fortran/114141] " jvdelisle at gcc dot gnu.org
  2024-02-28  2:21 ` jvdelisle at gcc dot gnu.org
@ 2024-02-28 15:43 ` kargl at gcc dot gnu.org
  2024-02-28 17:30 ` mikael at gcc dot gnu.org
                   ` (12 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: kargl at gcc dot gnu.org @ 2024-02-28 15:43 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

kargl at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4

--- Comment #3 from kargl at gcc dot gnu.org ---
(In reply to Jerry DeLisle from comment #2)
> It looks like the 'selector' in this case is an expr.
> 
> The expr must be a pointer object or a 'designator'
> 
> A designator must be:
> 
> R901
> designator
> 
> object-name
> array-element
> array-section
> coindexed-named-object
> complex-part-designator
> structure-component
> substring
> 
> I am not seeing the expr in the example as one of these listed. ???

Yep, agreed.  I went back an re-read the section about ASSOCIATE.
Not sure how I convinced myself that a constant expression, which
reduces to a constant is okay.

I suppose the question is "do we generate a better error message
or simply close the PR?"

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2024-02-28 15:43 ` kargl at gcc dot gnu.org
@ 2024-02-28 17:30 ` mikael at gcc dot gnu.org
  2024-02-28 18:13 ` kargl at gcc dot gnu.org
                   ` (11 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: mikael at gcc dot gnu.org @ 2024-02-28 17:30 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

Mikael Morin <mikael at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mikael at gcc dot gnu.org

--- Comment #4 from Mikael Morin <mikael at gcc dot gnu.org> ---
(In reply to Jerry DeLisle from comment #2)
> It looks like the 'selector' in this case is an expr.
> 
Agreed.

> The expr must be a pointer object or a 'designator'
> 
Really?  Can't find the constraint saying that.
The only associate constraint mentioning 'designator' I can see is this:

  C1105 (R1105) expr shall not be a designator of a procedure pointer or a
                function reference that returns a procedure pointer.

... but it relates more to procedure pointers (and it is a 'shall NOT be'
constraint).


(In reply to kargl from comment #3)
> Yep, agreed.  I went back an re-read the section about ASSOCIATE.
> Not sure how I convinced myself that a constant expression, which
> reduces to a constant is okay.
> 
Not sure how you convinced yourself it isn't. ;-)

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2024-02-28 17:30 ` mikael at gcc dot gnu.org
@ 2024-02-28 18:13 ` kargl at gcc dot gnu.org
  2024-02-28 19:27 ` mikael at gcc dot gnu.org
                   ` (10 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: kargl at gcc dot gnu.org @ 2024-02-28 18:13 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

kargl at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P4                          |P5
           Severity|normal                      |enhancement

--- Comment #5 from kargl at gcc dot gnu.org ---
(In reply to Mikael Morin from comment #4)

> (In reply to kargl from comment #3)
> > Yep, agreed.  I went back an re-read the section about ASSOCIATE.
> > Not sure how I convinced myself that a constant expression, which
> > reduces to a constant is okay.
> > 
> Not sure how you convinced yourself it isn't. ;-)

x => log(cmplx(-1,0))

R1104 association  is associate-name => selector

R1105 selector     is expr
                   or variable

R902 variable      is designator
                   or function-reference

R901 designator    is object-name
                   or array-element
                   or array-section
                   or coindexed-named-object
                   or complex-part-designator
                   or structure-component
                   or substring

log(cmplx(-1,0)) is certainly not a designator.

log(cmplx(-1,0)) is a function-reference.  But this then
leads to

C902 (R902) function-reference shall have a data pointer result.


log(cmplx(-1,0)) violates C902, so this then means that it
must be an expr.  One now needs


R915 complex-part-designator  is designator % RE
                              or designator % IM

C922 (R915) The designator shall be of complex type.

which shows that expr%im is invalid; even though log(cmplx(-1,0))
reduces to a constant (i.e., it's not a named constant.  This
is likely the error [pun intended] in my ways.).

Sometimes the trees get in the way of seeing the forest.


Arguably, the error message is wrong
gfortran13 -c a.f90
a.f90:6:13:

    6 |       y = x%im
      |             1
Error: Symbol 'x' at (1) has no IMPLICIT type

'x' has the type of COMPLEX.

The version of the code where Jerry wraps log(cmplx(-1,0))
in parentheses.  Generates a better error message


$ gfc pr114141.f90
pr114141.f90:6:14:

    6 |       y = x%im
      |              1
Error: The RE or IM part_ref at (1) must be applied to a COMPLEX expression

but this is still wrong in that RE and IM are applied to a designator.

I'll leave the PR open has an enhancement request.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2024-02-28 18:13 ` kargl at gcc dot gnu.org
@ 2024-02-28 19:27 ` mikael at gcc dot gnu.org
  2024-02-28 20:24 ` sgk at troutmask dot apl.washington.edu
                   ` (9 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: mikael at gcc dot gnu.org @ 2024-02-28 19:27 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #6 from Mikael Morin <mikael at gcc dot gnu.org> ---
(In reply to kargl from comment #5)
> (In reply to Mikael Morin from comment #4)
> 
> > (In reply to kargl from comment #3)
> > > Yep, agreed.  I went back an re-read the section about ASSOCIATE.
> > > Not sure how I convinced myself that a constant expression, which
> > > reduces to a constant is okay.
> > > 
> > Not sure how you convinced yourself it isn't. ;-)
> 
> x => log(cmplx(-1,0))
> 
> R1104 association  is associate-name => selector
> 
> R1105 selector     is expr
>                    or variable
> 
> R902 variable      is designator
>                    or function-reference
> 
> R901 designator    is object-name
>                    or array-element
>                    or array-section
>                    or coindexed-named-object
>                    or complex-part-designator
>                    or structure-component
>                    or substring
> 
> log(cmplx(-1,0)) is certainly not a designator.
> 
> log(cmplx(-1,0)) is a function-reference.  But this then
> leads to
> 
> C902 (R902) function-reference shall have a data pointer result.
> 
> 
> log(cmplx(-1,0)) violates C902, so this then means that it
> must be an expr.
Agreed.

>  One now needs
> 
> 
> R915 complex-part-designator  is designator % RE
>                               or designator % IM
> 
> C922 (R915) The designator shall be of complex type.
> 
> which shows that expr%im is invalid; even though log(cmplx(-1,0))
> reduces to a constant (i.e., it's not a named constant.  This
> is likely the error [pun intended] in my ways.).
> 
This is about x%im, which is a different expression from log(cmplx(-1, 0)).
x is an associate-name, and thus (I think) an object-name, and a valid
designator, even if it's associated selector isn't.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2024-02-28 19:27 ` mikael at gcc dot gnu.org
@ 2024-02-28 20:24 ` sgk at troutmask dot apl.washington.edu
  2024-02-28 20:42 ` sgk at troutmask dot apl.washington.edu
                   ` (8 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: sgk at troutmask dot apl.washington.edu @ 2024-02-28 20:24 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #7 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Wed, Feb 28, 2024 at 07:27:24PM +0000, mikael at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141
> 
> --- Comment #6 from Mikael Morin <mikael at gcc dot gnu.org> ---
> (In reply to kargl from comment #5)
> > (In reply to Mikael Morin from comment #4)
> > 
> > > (In reply to kargl from comment #3)
> > > > Yep, agreed.  I went back an re-read the section about ASSOCIATE.
> > > > Not sure how I convinced myself that a constant expression, which
> > > > reduces to a constant is okay.
> > > > 
> > > Not sure how you convinced yourself it isn't. ;-)
> > 
> > x => log(cmplx(-1,0))
> > 
> > R1104 association  is associate-name => selector
> > 
> > R1105 selector     is expr
> >                    or variable
> > 
> > R902 variable      is designator
> >                    or function-reference
> > 
> > R901 designator    is object-name
> >                    or array-element
> >                    or array-section
> >                    or coindexed-named-object
> >                    or complex-part-designator
> >                    or structure-component
> >                    or substring
> > 
> > log(cmplx(-1,0)) is certainly not a designator.
> > 
> > log(cmplx(-1,0)) is a function-reference.  But this then
> > leads to
> > 
> > C902 (R902) function-reference shall have a data pointer result.
> > 
> > 
> > log(cmplx(-1,0)) violates C902, so this then means that it
> > must be an expr.
> Agreed.
> 
> >  One now needs
> > 
> > 
> > R915 complex-part-designator  is designator % RE
> >                               or designator % IM
> > 
> > C922 (R915) The designator shall be of complex type.
> > 
> > which shows that expr%im is invalid; even though log(cmplx(-1,0))
> > reduces to a constant (i.e., it's not a named constant.  This
> > is likely the error [pun intended] in my ways.).
> > 
> This is about x%im, which is a different expression from log(cmplx(-1, 0)).
> x is an associate-name, and thus (I think) an object-name, and a valid
> designator, even if it's associated selector isn't.
> 

Indeed.  Bit more reading of F2023, 11.1.3 agrees with you.

   11.1.3.1

   The ASSOCIATE construct associates named entities with expressions
   or variables during the execution of its block.  These named construct
   entities (19.4) are associating entities (19.5.1.6).  The names are
   associate names.

   11.1.3.3(5) The associating entity itself is a variable, but ...

The "but ..." applies to whether the selector is a definable variable.

So, 'y = x%im' is allowed, but 'x%im = 42' is disallowed because
the selector is not definable.  Interesting twist.  This then 
suggests that Jerry's use of parentheses should be accepted.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (6 preceding siblings ...)
  2024-02-28 20:24 ` sgk at troutmask dot apl.washington.edu
@ 2024-02-28 20:42 ` sgk at troutmask dot apl.washington.edu
  2024-02-28 22:32 ` jvdelisle at gcc dot gnu.org
                   ` (7 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: sgk at troutmask dot apl.washington.edu @ 2024-02-28 20:42 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #8 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Wed, Feb 28, 2024 at 08:24:16PM +0000, sgk at troutmask dot
apl.washington.edu wrote:
> 
> Indeed.  Bit more reading of F2023, 11.1.3 agrees with you.
> 
>    11.1.3.1
> 
>    The ASSOCIATE construct associates named entities with expressions
>    or variables during the execution of its block.  These named construct
>    entities (19.4) are associating entities (19.5.1.6).  The names are
>    associate names.
> 
>    11.1.3.3(5) The associating entity itself is a variable, but ...
> 
> The "but ..." applies to whether the selector is a definable variable.
> 
> So, 'y = x%im' is allowed, but 'x%im = 42' is disallowed because
> the selector is not definable.  Interesting twist.  This then 
> suggests that Jerry's use of parentheses should be accepted.
> 

As a quick follow-up.

program p
   associate(x => sin(cmplx(0.5,0.5)))
      print *, x
      print *, x%im  ! <-- allowed with my patch
      x%im = 42      ! <-- this is an error
      print *, x
   end associate
end

% gfcx -o z a.f90
a.f90:5:6:

    5 |       x%im = 42
      |      1
Error: 'x' at (1) associated to expression cannot be used in
a variable definition context (assignment)

Mikael, thanks for the feedback.  I'll see if I can fix
the parentheses case this weekend.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (7 preceding siblings ...)
  2024-02-28 20:42 ` sgk at troutmask dot apl.washington.edu
@ 2024-02-28 22:32 ` jvdelisle at gcc dot gnu.org
  2024-02-29 18:33 ` pault at gcc dot gnu.org
                   ` (6 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: jvdelisle at gcc dot gnu.org @ 2024-02-28 22:32 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #9 from Jerry DeLisle <jvdelisle at gcc dot gnu.org> ---
--- snip ---
> % gfcx -o z a.f90
> a.f90:5:6:
> 
>     5 |       x%im = 42
>       |      1
> Error: 'x' at (1) associated to expression cannot be used in
> a variable definition context (assignment)
> 
> Mikael, thanks for the feedback.  I'll see if I can fix
> the parentheses case this weekend.

This is definitely a 42 case, which is why I had three '?' in my reply.

And if you understand this, you are OK in my book. :)

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (8 preceding siblings ...)
  2024-02-28 22:32 ` jvdelisle at gcc dot gnu.org
@ 2024-02-29 18:33 ` pault at gcc dot gnu.org
  2024-02-29 20:25 ` sgk at troutmask dot apl.washington.edu
                   ` (5 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pault at gcc dot gnu.org @ 2024-02-29 18:33 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

Paul Thomas <pault at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
                 CC|                            |pault at gcc dot gnu.org
   Last reconfirmed|                            |2024-02-29

--- Comment #10 from Paul Thomas <pault at gcc dot gnu.org> ---
(In reply to Jerry DeLisle from comment #9)
> --- snip ---
> > % gfcx -o z a.f90
> > a.f90:5:6:
> > 
> >     5 |       x%im = 42
> >       |      1
> > Error: 'x' at (1) associated to expression cannot be used in
> > a variable definition context (assignment)
> > 
> > Mikael, thanks for the feedback.  I'll see if I can fix
> > the parentheses case this weekend.
> 
> This is definitely a 42 case, which is why I had three '?' in my reply.
> 
> And if you understand this, you are OK in my book. :)

BTW Both nagfor and ifort compile the testcase without complaint.

Your fix is closely related to my patch for not-yet-parsed function selectors.

Cheers

Paul

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (9 preceding siblings ...)
  2024-02-29 18:33 ` pault at gcc dot gnu.org
@ 2024-02-29 20:25 ` sgk at troutmask dot apl.washington.edu
  2024-03-01  9:15 ` pault at gcc dot gnu.org
                   ` (4 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: sgk at troutmask dot apl.washington.edu @ 2024-02-29 20:25 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #11 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
On Thu, Feb 29, 2024 at 06:33:51PM +0000, pault at gcc dot gnu.org wrote:
> --- Comment #10 from Paul Thomas <pault at gcc dot gnu.org> ---
> (In reply to Jerry DeLisle from comment #9)
> > --- snip ---
> > > % gfcx -o z a.f90
> > > a.f90:5:6:
> > > 
> > >     5 |       x%im = 42
> > >       |      1
> > > Error: 'x' at (1) associated to expression cannot be used in
> > > a variable definition context (assignment)
> > > 
> > > Mikael, thanks for the feedback.  I'll see if I can fix
> > > the parentheses case this weekend.
> > 
> > This is definitely a 42 case, which is why I had three '?' in my reply.
> > 
> > And if you understand this, you are OK in my book. :)
> 
> BTW Both nagfor and ifort compile the testcase without complaint.
> 
> Your fix is closely related to my patch for not-yet-parsed function selectors.
> 

I know you had some ASSOCIATE patches in the works, and
certainly do not want to interfere.  Do you want to
incorporate my patch or some variation into your work?
I'm hoping to take a stab at the issue Jerry raised 
with parentheses this weekend.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (10 preceding siblings ...)
  2024-02-29 20:25 ` sgk at troutmask dot apl.washington.edu
@ 2024-03-01  9:15 ` pault at gcc dot gnu.org
  2024-03-01  9:16 ` pault at gcc dot gnu.org
                   ` (3 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pault at gcc dot gnu.org @ 2024-03-01  9:15 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #12 from Paul Thomas <pault at gcc dot gnu.org> ---
(In reply to Steve Kargl from comment #11)
...snip...
> I know you had some ASSOCIATE patches in the works, and
> certainly do not want to interfere.  Do you want to
> incorporate my patch or some variation into your work?
> I'm hoping to take a stab at the issue Jerry raised 
> with parentheses this weekend.

Hi Steve,

Interference was not what I had in mind :-)

I was thinking of breaking the patch
https://gcc.gnu.org/pipermail/fortran/2024-January/060092.html in two; the
first to deal with derived type functions and the second for class functions.
Your patch for this PR would sit nicely in the first.

Cheers

Paul

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (11 preceding siblings ...)
  2024-03-01  9:15 ` pault at gcc dot gnu.org
@ 2024-03-01  9:16 ` pault at gcc dot gnu.org
  2024-03-02 17:32 ` pault at gcc dot gnu.org
                   ` (2 subsequent siblings)
  15 siblings, 0 replies; 17+ messages in thread
From: pault at gcc dot gnu.org @ 2024-03-01  9:16 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #13 from Paul Thomas <pault at gcc dot gnu.org> ---
(In reply to Steve Kargl from comment #11)
...snip...
> I know you had some ASSOCIATE patches in the works, and
> certainly do not want to interfere.  Do you want to
> incorporate my patch or some variation into your work?
> I'm hoping to take a stab at the issue Jerry raised 
> with parentheses this weekend.

Hi Steve,

Interference was not what I had in mind :-)

I was thinking of breaking the patch
https://gcc.gnu.org/pipermail/fortran/2024-January/060092.html in two; the
first to deal with derived type functions and the second for class functions.
Your patch for this PR would sit nicely in the first.

Cheers

Paul

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (12 preceding siblings ...)
  2024-03-01  9:16 ` pault at gcc dot gnu.org
@ 2024-03-02 17:32 ` pault at gcc dot gnu.org
  2024-03-15  6:53 ` cvs-commit at gcc dot gnu.org
  2024-03-15  7:09 ` pault at gcc dot gnu.org
  15 siblings, 0 replies; 17+ messages in thread
From: pault at gcc dot gnu.org @ 2024-03-02 17:32 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #14 from Paul Thomas <pault at gcc dot gnu.org> ---
To fix the parentheses wrinkle, this works:
diff --git a/gcc/fortran/match.cc b/gcc/fortran/match.cc
index eee569dac91..64f61c50c66 100644
--- a/gcc/fortran/match.cc
+++ b/gcc/fortran/match.cc
@@ -1963,6 +1963,20 @@ gfc_match_associate (void)
          goto assocListError;
        }

+      /* If the selector expression is enclosed in parentheses and the
+        expression is not a variable, throw the parentheses away.  */
+      while (newAssoc->target->expr_type == EXPR_OP
+            && newAssoc->target->value.op.op == INTRINSIC_PARENTHESES)
+       {
+         if (newAssoc->target->value.op.op1->expr_type == EXPR_VARIABLE)
+           break;
+         else
+           {
+             gfc_expr *e = gfc_copy_expr (newAssoc->target->value.op.op1);
+             gfc_replace_expr (newAssoc->target, e);
+           }
+       }
+
       /* The `variable' field is left blank for now; because the target is not

To maintain compatibility with
https://gcc.gnu.org/pipermail/fortran/2024-January/060092.html:

@@ -2220,7 +2235,8 @@ gfc_match_varspec (gfc_expr *primary, int equiv_flag,
bool sub_flag,
                        || tgt_expr->symtree->n.sym->attr.if_source
                                                                == IFSRC_DECL);
       permissible = permissible
-                   || (tgt_expr && tgt_expr->expr_type == EXPR_OP);
+                   || (tgt_expr && (tgt_expr->expr_type == EXPR_OP
+                       || (inquiry && tgt_expr->expr_type == EXPR_FUNCTION)));

       if (permissible)
        {

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (13 preceding siblings ...)
  2024-03-02 17:32 ` pault at gcc dot gnu.org
@ 2024-03-15  6:53 ` cvs-commit at gcc dot gnu.org
  2024-03-15  7:09 ` pault at gcc dot gnu.org
  15 siblings, 0 replies; 17+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2024-03-15  6:53 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

--- Comment #15 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Paul Thomas <pault@gcc.gnu.org>:

https://gcc.gnu.org/g:3fd46d859cda1074125449a4cc680ce59fcebc38

commit r14-9489-g3fd46d859cda1074125449a4cc680ce59fcebc38
Author: Paul Thomas <pault@gcc.gnu.org>
Date:   Fri Mar 15 06:52:59 2024 +0000

    Fortran: Fix class/derived/complex function associate selectors [PR87477]

    2024-03-15  Paul Thomas  <pault@gcc.gnu.org>

    gcc/fortran
            PR fortran/87477
            PR fortran/89645
            PR fortran/99065
            PR fortran/114141
            PR fortran/114280
            * class.cc (gfc_change_class): New function needed for
            associate names, when rank changes or a derived type is
            produced by resolution
            * dump-parse-tree.cc (show_code_node): Make output for SELECT
            TYPE more comprehensible.
            * expr.cc (find_inquiry_ref): Do not simplify expressions of
            an inferred type.
            * gfortran.h : Add 'gfc_association_list' to structure
            'gfc_association_list'. Add prototypes for
            'gfc_find_derived_types', 'gfc_fixup_inferred_type_refs' and
            'gfc_change_class'. Add macro IS_INFERRED_TYPE.
            * match.cc (copy_ts_from_selector_to_associate): Add bolean arg
            'select_type' with default false. If this is a select type name
            and the selector is a inferred type, build the class type and
            apply it to the associate name.
            (build_associate_name): Pass true to 'select_type' in call to
            previous.
            * parse.cc (parse_associate): If the selector is inferred type
            the associate name is too. Make sure that function selector
            class and rank, if known, are passed to the associate name. If
            a function result exists, pass its typespec to the associate
            name.
            * primary.cc (resolvable_fcns): New function to check that all
            the function references are resolvable.
            (gfc_match_varspec): If a scalar derived type select type
            temporary has an array reference, match the array reference,
            treating this in the same way as an equivalence member. Do not
            set 'inquiry' if applied to an unknown type the inquiry name
            is ambiguous with the component of an accessible derived type.
            Check that resolution of the target expression is OK by testing
            if the symbol is declared or is an operator expression, then
            using 'resolvable_fcns' recursively. If all is well, resolve
            the expression. If this is an inferred type with a component
            reference, call 'gfc_find_derived_types' to find a suitable
            derived type. If there is an inquiry ref and the symbol either
            is of unknown type or is inferred to be a derived type, set the
            primary and symbol TKR appropriately.
            * resolve.cc (resolve_variable): Call new function below.
            (gfc_fixup_inferred_type_refs): New function to ensure that the
            expression references for a inferred type are consistent with
            the now fixed up selector.
            (resolve_assoc_var): Ensure that derived type or class function
            selectors transmit the correct arrayspec to the associate name.
            (resolve_select_type): If the selector is an associate name of
            inferred type and has no component references, the associate
            name should have its typespec. Simplify the conversion of a
            class array to class scalar by calling 'gfc_change_class'.
            Make sure that a class, inferred type selector with an array
            ref transfers the typespec from the symbol to the expression.
            * symbol.cc (gfc_set_default_type): If an associate name with
            unknown type has a selector expression, try resolving the expr.
            (find_derived_types, gfc_find_derived_types): New functions
            that search for a derived type with a given name.
            * trans-expr.cc (gfc_conv_variable): Some inferred type exprs
            escape resolution so call 'gfc_fixup_inferred_type_refs'.
            * trans-stmt.cc (trans_associate_var): Tidy up expression for
            'class_target'. Finalize and free class function results.
            Correctly handle selectors that are class functions and class
            array references, passed as derived types.

    gcc/testsuite/
            PR fortran/87477
            PR fortran/89645
            PR fortran/99065
            * gfortran.dg/associate_64.f90 : New test
            * gfortran.dg/associate_66.f90 : New test
            * gfortran.dg/associate_67.f90 : New test

            PR fortran/114141
            * gfortran.dg/associate_65.f90 : New test

            PR fortran/114280
            * gfortran.dg/associate_68.f90 : New test

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [Bug fortran/114141] ASSOCIATE and complex part ref when associate target is a function
  2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
                   ` (14 preceding siblings ...)
  2024-03-15  6:53 ` cvs-commit at gcc dot gnu.org
@ 2024-03-15  7:09 ` pault at gcc dot gnu.org
  15 siblings, 0 replies; 17+ messages in thread
From: pault at gcc dot gnu.org @ 2024-03-15  7:09 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114141

Paul Thomas <pault at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #16 from Paul Thomas <pault at gcc dot gnu.org> ---
This mega-patch, on the scale of the importance of the problem, was required
because of gfortran's one pass parsing. It might be a temporary fix because I
am contemplating how an initial pass of contained procedures might be
introduced.

Fixed on mainline.

Paul

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2024-03-15  7:09 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-27 22:11 [Bug fortran/114141] New: ASSOCIATE and complex part ref when associate target is a function kargl at gcc dot gnu.org
2024-02-28  2:11 ` [Bug fortran/114141] " jvdelisle at gcc dot gnu.org
2024-02-28  2:21 ` jvdelisle at gcc dot gnu.org
2024-02-28 15:43 ` kargl at gcc dot gnu.org
2024-02-28 17:30 ` mikael at gcc dot gnu.org
2024-02-28 18:13 ` kargl at gcc dot gnu.org
2024-02-28 19:27 ` mikael at gcc dot gnu.org
2024-02-28 20:24 ` sgk at troutmask dot apl.washington.edu
2024-02-28 20:42 ` sgk at troutmask dot apl.washington.edu
2024-02-28 22:32 ` jvdelisle at gcc dot gnu.org
2024-02-29 18:33 ` pault at gcc dot gnu.org
2024-02-29 20:25 ` sgk at troutmask dot apl.washington.edu
2024-03-01  9:15 ` pault at gcc dot gnu.org
2024-03-01  9:16 ` pault at gcc dot gnu.org
2024-03-02 17:32 ` pault at gcc dot gnu.org
2024-03-15  6:53 ` cvs-commit at gcc dot gnu.org
2024-03-15  7:09 ` pault at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).