public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug target/26525] New: missed opportunity to use value in fp reg
@ 2006-03-02 2:40 amodra at bigpond dot net dot au
2006-03-02 3:07 ` [Bug target/26525] " pinskia at gcc dot gnu dot org
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: amodra at bigpond dot net dot au @ 2006-03-02 2:40 UTC (permalink / raw)
To: gcc-bugs
The following compiled with -m64 -O2 -S
double f1 (double xh)
{
long long hi = (long long) xh;
if (hi == __LONG_LONG_MAX__)
{
xh += -__LONG_LONG_MAX__ - 1;
xh += 1.0;
}
else
xh -= hi;
return xh;
}
generates
fctidz 0,1
li 0,-1
rldicl 0,0,0,1
stfd 0,-16(1)
ld 9,-16(1)
cmpd 7,9,0
beq 7,.L7
std 9,-16(1) #
nop #
nop #
nop #
lfd 13,-16(1) #
fcfid 0,13 #
fsub 1,1,0
blr
.p2align 4,,15
.L7:
lfs 0,.LC1@toc(2)
lfs 13,.LC3@toc(2)
fsub 0,1,0
fadd 1,0,13
blr
The sequence marked with # could be replaced with just one insn,
fcfid 0,0
because HI is in fr0 at that point.
A similar function
double f2 (double xh)
{
long long hi = (long long) xh;
xh -= hi;
if (hi == __LONG_LONG_MAX__)
{
xh += hi;
xh += -__LONG_LONG_MAX__ - 1;
xh += 1.0;
}
return xh;
}
does use the value in th fp reg
fctidz 0,1
li 0,-1
rldicl 0,0,0,1
stfd 0,-16(1)
fcfid 0,0 <== here
fsub 1,1,0
ld 9,-16(1)
cmpd 7,9,0
bnelr 7
fadd 0,0,1
lfs 13,.LC5@toc(2)
lfs 12,.LC7@toc(2)
fsub 0,0,13
fadd 1,0,12
blr
--
Summary: missed opportunity to use value in fp reg
Product: gcc
Version: 4.2.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: amodra at bigpond dot net dot au
GCC target triplet: powerpc64-linux
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525
^ permalink raw reply [flat|nested] 6+ messages in thread* [Bug target/26525] missed opportunity to use value in fp reg 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au @ 2006-03-02 3:07 ` pinskia at gcc dot gnu dot org 2006-06-23 1:41 ` pinskia at gcc dot gnu dot org ` (3 subsequent siblings) 4 siblings, 0 replies; 6+ messages in thread From: pinskia at gcc dot gnu dot org @ 2006-03-02 3:07 UTC (permalink / raw) To: gcc-bugs ------- Comment #1 from pinskia at gcc dot gnu dot org 2006-03-02 03:07 ------- This is definitely a RA issue. (insn:HI 12 65 15 2 (set (reg/v:DI 121 [ hi ]) (fix:DI (reg/v:DF 123 [ xh ]))) 265 {fix_truncdfdi2} (insn_list:REG_DEP_TRUE 6 (nil)) (nil)) (insn:HI 15 12 16 2 (set (reg:CC 125) (compare:CC (reg/v:DI 121 [ hi ]) (reg:DI 124))) 412 {*cmpdi_internal1} (insn_list:REG_DEP_TRUE 65 (insn_list:REG_DEP_TRUE 12 (nil))) (expr_list:REG_DEAD (reg:DI 124) (expr_list:REG_EQUAL (compare:CC (reg/v:DI 121 [ hi ]) (const_int 9223372036854775807 [0x7fffffffffffffff])) (nil)))) ... (insn:HI 36 34 37 4 (set (reg:DF 132) (float:DF (reg/v:DI 121 [ hi ]))) 262 {floatdidf2} (nil) (expr_list:REG_DEAD (reg/v:DI 121 [ hi ]) (nil))) The problem is that we need to put 121 into an integer register for the compare but also in a fp register for the int-to-float. Confirmed. -- pinskia at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |enhancement Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 Keywords| |missed-optimization, ra Last reconfirmed|0000-00-00 00:00:00 |2006-03-02 03:07:41 date| | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/26525] missed opportunity to use value in fp reg 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au 2006-03-02 3:07 ` [Bug target/26525] " pinskia at gcc dot gnu dot org @ 2006-06-23 1:41 ` pinskia at gcc dot gnu dot org 2007-10-23 17:13 ` pinskia at gcc dot gnu dot org ` (2 subsequent siblings) 4 siblings, 0 replies; 6+ messages in thread From: pinskia at gcc dot gnu dot org @ 2006-06-23 1:41 UTC (permalink / raw) To: gcc-bugs ------- Comment #2 from pinskia at gcc dot gnu dot org 2006-06-23 01:41 ------- I knew when I filed PR 27946, I saw it before. The reason why I am closing this one as a dup of that is because PR 27946 has a simple testcase and mentions that YARA already fixes the problem. *** This bug has been marked as a duplicate of 27946 *** -- pinskia at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |DUPLICATE http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/26525] missed opportunity to use value in fp reg 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au 2006-03-02 3:07 ` [Bug target/26525] " pinskia at gcc dot gnu dot org 2006-06-23 1:41 ` pinskia at gcc dot gnu dot org @ 2007-10-23 17:13 ` pinskia at gcc dot gnu dot org 2008-09-14 3:28 ` pinskia at gcc dot gnu dot org 2009-05-14 3:19 ` bje at gcc dot gnu dot org 4 siblings, 0 replies; 6+ messages in thread From: pinskia at gcc dot gnu dot org @ 2007-10-23 17:13 UTC (permalink / raw) To: gcc-bugs ------- Comment #3 from pinskia at gcc dot gnu dot org 2007-10-23 17:13 ------- Actually this is not fully the same issue as the other PR as I can make the other PR work if I use "?f#r" as the constaint but not this one. So reope -- pinskia at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|DUPLICATE | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/26525] missed opportunity to use value in fp reg 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au ` (2 preceding siblings ...) 2007-10-23 17:13 ` pinskia at gcc dot gnu dot org @ 2008-09-14 3:28 ` pinskia at gcc dot gnu dot org 2009-05-14 3:19 ` bje at gcc dot gnu dot org 4 siblings, 0 replies; 6+ messages in thread From: pinskia at gcc dot gnu dot org @ 2008-09-14 3:28 UTC (permalink / raw) To: gcc-bugs ------- Comment #4 from pinskia at gcc dot gnu dot org 2008-09-14 03:27 ------- This still happens with IRA. -- pinskia at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEW http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/26525] missed opportunity to use value in fp reg 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au ` (3 preceding siblings ...) 2008-09-14 3:28 ` pinskia at gcc dot gnu dot org @ 2009-05-14 3:19 ` bje at gcc dot gnu dot org 4 siblings, 0 replies; 6+ messages in thread From: bje at gcc dot gnu dot org @ 2009-05-14 3:19 UTC (permalink / raw) To: gcc-bugs ------- Comment #5 from bje at gcc dot gnu dot org 2009-05-14 03:19 ------- Still present in GCC: (GNU) 4.5.0 20090513 (experimental) [trunk revision 147498]. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26525 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-05-14 3:19 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2006-03-02 2:40 [Bug target/26525] New: missed opportunity to use value in fp reg amodra at bigpond dot net dot au 2006-03-02 3:07 ` [Bug target/26525] " pinskia at gcc dot gnu dot org 2006-06-23 1:41 ` pinskia at gcc dot gnu dot org 2007-10-23 17:13 ` pinskia at gcc dot gnu dot org 2008-09-14 3:28 ` pinskia at gcc dot gnu dot org 2009-05-14 3:19 ` bje at gcc dot gnu dot 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).