middle-end/7198: ia64.md missing usual fnma patterns

middle-end/7198: ia64.md missing usual fnma patterns

[tprince]

>Number:         7198
>Category:       middle-end
>Synopsis:       ia64.md missing usual fnma patterns
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Wed Jul 03 08:36:01 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     Tim Prince
>Release:        3.1.1 20020701
>Organization:
>Environment:
red hat 7.2
>Description:
fnma patterns in ia64.md don't match with normal source code so generated code always uses separate multiply and subtract
>How-To-Repeat:
gcc -O2 -S -funroll-loops -fforce-addr kcs.c
shows several missed opportunities to employ fnma instructions
>Fix:
patch included for ia64.md which adds fnma patterns for SF, DF, and TF
>Release-Note:
>Audit-Trail:
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/x-gzip-compressed; name="ia64_fnma.diff.gz"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="ia64_fnma.diff.gz"

H4sICBgUIz0AA2lhNjRfZm5tYS5kaWZmAMWUa4ubQBSGPye/4iAEoo3rdV3rsoQFtaS0W2j9UCjL
MPWSDHgJzvilv75nTHbrYpJNQ8sKjp4z875zPD6MpmnAqOdeVdkk2XRwv23B8sE2g2szcFx8Me2p
rutgbJoqNwSrDN6mxjpNdTlleqYjAyNt6oKtDWllPPt1OXzsSgAbzJvA9YPeGP20l5eMwb72rQUO
LvQpueU+9d4FjPQpwO0tLJdLWHFgArYN5+xnmYNoIKV1U7OUluwXxhvGgXKYV6zu8NHmaxWDrhSq
ukSX6TuYZ3nB6pywmtegKJgB+DHnucB1VKQb0mzzltZZ8C0GE5SiJWjCuMjbpxkFlLtCUVE52e0j
l47FzpOYNC0ptqY10BcfdvrJrjhc3kcTrGVsZB2rYl/EEZl9QobXY//l+wYoRV3RKw4zE+5gZi1g
ZuMdO4P2ECpECwoTtGZdRdKSci7NKpoq6qMsBPv7srkammYZL1wFpy7t8kFgfM9Z4OANgPFvZMr3
3g6Y8HxgwhEw4d8BEx4FJrwMmPASYLL/Akx2EphXu3wIGMexrAUOgxPGwRBT7hueMMn5wCQjYJIe
GFGcx0vyzMvYxjrD5qDQPi38w8zqIYk+keh7Ej2EUUhWURSR+MvXz/fJgKY9S7GEKf53NImTNL36
C34DP4s0TCUHAAA=

Re: middle-end/7198: ia64.md missing usual fnma patterns

[Steven Bosscher]

The following reply was made to PR target/7198; it has been noted by GNATS.

From: Steven Bosscher <s.bosscher@student.tudelft.nl>
To: tprince@computer.org
Cc: gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org
Subject: Re: middle-end/7198: ia64.md missing usual fnma patterns
Date: 01 Mar 2003 15:44:36 +0100

 Op za 01-03-2003, om 15:28 schreef Tim Prince:
 > On Saturday 01 March 2003 04:01, Steven Bosscher wrote:
 > > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&
 > >pr=7198
 > >
 > > Does this bug still exist in the 3.3 branch and mainline?  It looks like
 > > the patterns from your patch are already in ia64.md.
 > That's interesting; I don't find them in the gcc-20030224 (3.3) snapshot.  In 
 > fact, I just checked this week, and that snapshot was not employing fnma 
 > instructions in my test code, so I applied this patch again.  The patterns 
 > which came in ia64.md require a sign change, to match the more common source 
 > code pattern, so it seems that a pattern with the sign change already applied 
 > is needed.
 
 Ah, you're right, I overlooked the "neg:[SDT]F" part" in ia64.md.
 
 

Re: middle-end/7198: ia64.md missing usual fnma patterns

[Tim Prince]

The following reply was made to PR target/7198; it has been noted by GNATS.

From: Tim Prince <timothyprince@sbcglobal.net>
To: Steven Bosscher <s.bosscher@student.tudelft.nl>, gcc-gnats@gcc.gnu.org,
   gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org, gcc-prs@gcc.gnu.org,
   tprince@computer.org
Cc:  
Subject: Re: middle-end/7198: ia64.md missing usual fnma patterns
Date: Sat, 1 Mar 2003 06:28:31 -0800

 On Saturday 01 March 2003 04:01, Steven Bosscher wrote:
 > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&
 >pr=7198
 >
 > Does this bug still exist in the 3.3 branch and mainline?  It looks like
 > the patterns from your patch are already in ia64.md.
 >
 > Greetz
 > Steven
 
 That's interesting; I don't find them in the gcc-20030224 (3.3) snapshot.  In 
 fact, I just checked this week, and that snapshot was not employing fnma 
 instructions in my test code, so I applied this patch again.  The patterns 
 which came in ia64.md require a sign change, to match the more common source 
 code pattern, so it seems that a pattern with the sign change already applied 
 is needed.
 I don't find my patterns in the gcc-3.2.2 either, so I just tried applying 
 the patch (on my ia32 laptop here at home), and it appears to take 
 successfully.
 My co-workers have not found a way to access CVS from our ia64 machines at 
 the office, so I have always dealt with snapshots.
 Thanks for looking at this. Not exactly a bug, it's a missing optimization.
 -- 
 Tim Prince

Re: middle-end/7198: ia64.md missing usual fnma patterns

[Steven Bosscher]

The following reply was made to PR target/7198; it has been noted by GNATS.

From: Steven Bosscher <s.bosscher@student.tudelft.nl>
To: gcc-gnats@gcc.gnu.org, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org,
	gcc-prs@gcc.gnu.org, tprince@computer.org
Cc:  
Subject: Re: middle-end/7198: ia64.md missing usual fnma patterns
Date: Sat, 01 Mar 2003 13:01:06 +0100

 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7198
 
 Does this bug still exist in the 3.3 branch and mainline?  It looks like 
 the patterns from your patch are already in ia64.md.
 
 Greetz
 Steven