public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/61098] New: Poor code setting count register for large loops
@ 2014-05-07 14:59 amodra at gmail dot com
2014-05-07 15:00 ` [Bug target/61098] " amodra at gmail dot com
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: amodra at gmail dot com @ 2014-05-07 14:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=61098
Bug ID: 61098
Summary: Poor code setting count register for large loops
Product: gcc
Version: 4.10.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: amodra at gmail dot com
Seen on a for (i = 0; i < 100000000; i++) loop
100003bc: 3d 20 05 f5 lis r9,1525
100003c0: 7d 29 03 a6 mtctr r9
100003c4: 7d 3e 1a 14 add r9,r30,r3
100003c8: 7d 49 02 a6 mfctr r10
100003cc: 61 4a e1 00 ori r10,r10,57600
100003d0: 7d 49 03 a6 mtctr r10
Note the horrible mtctr, mfctr, mtctr
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/61098] Poor code setting count register for large loops
2014-05-07 14:59 [Bug target/61098] New: Poor code setting count register for large loops amodra at gmail dot com
@ 2014-05-07 15:00 ` amodra at gmail dot com
2014-05-07 15:03 ` amodra at gmail dot com
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: amodra at gmail dot com @ 2014-05-07 15:00 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=61098
Alan Modra <amodra at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target| |powerpc64-linux,
| |powerpc64le-linux
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2014-05-07
Assignee|unassigned at gcc dot gnu.org |amodra at gmail dot com
Ever confirmed|0 |1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/61098] Poor code setting count register for large loops
2014-05-07 14:59 [Bug target/61098] New: Poor code setting count register for large loops amodra at gmail dot com
2014-05-07 15:00 ` [Bug target/61098] " amodra at gmail dot com
@ 2014-05-07 15:03 ` amodra at gmail dot com
2014-05-26 13:10 ` amodra at gcc dot gnu.org
2014-05-29 9:23 ` amodra at gmail dot com
3 siblings, 0 replies; 5+ messages in thread
From: amodra at gmail dot com @ 2014-05-07 15:03 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=61098
--- Comment #1 from Alan Modra <amodra at gmail dot com> ---
Created attachment 32754
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32754&action=edit
testcase
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/61098] Poor code setting count register for large loops
2014-05-07 14:59 [Bug target/61098] New: Poor code setting count register for large loops amodra at gmail dot com
2014-05-07 15:00 ` [Bug target/61098] " amodra at gmail dot com
2014-05-07 15:03 ` amodra at gmail dot com
@ 2014-05-26 13:10 ` amodra at gcc dot gnu.org
2014-05-29 9:23 ` amodra at gmail dot com
3 siblings, 0 replies; 5+ messages in thread
From: amodra at gcc dot gnu.org @ 2014-05-26 13:10 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61098
--- Comment #2 from Alan Modra <amodra at gcc dot gnu.org> ---
Author: amodra
Date: Mon May 26 13:09:48 2014
New Revision: 210932
URL: http://gcc.gnu.org/viewcvs?rev=210932&root=gcc&view=rev
Log:
PR target/61098
* config/rs6000/rs6000.c (rs6000_emit_set_const): Remove unneeded
params and return a bool. Remove dead code. Update comment.
Assert we have a const_int source. Remove bogus code from
32-bit HWI days. Move !TARGET_POWERPC64 handling, and correct
handling of constants > 2G and reg_equal note, from..
(rs6000_emit_set_long_const): ..here. Remove unneeded param and
return value. Update comment. If we can, use a new pseudo
for intermediate calculations.
* config/rs6000/rs6000-protos.h (rs6000_emit_set_const): Update
prototype.
* config/rs6000/rs6000.md (movsi_internal1_single+1): Update
call to rs6000_emit_set_const in splitter.
(movdi_internal64+2, +3): Likewise.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/rs6000/rs6000-protos.h
trunk/gcc/config/rs6000/rs6000.c
trunk/gcc/config/rs6000/rs6000.md
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug target/61098] Poor code setting count register for large loops
2014-05-07 14:59 [Bug target/61098] New: Poor code setting count register for large loops amodra at gmail dot com
` (2 preceding siblings ...)
2014-05-26 13:10 ` amodra at gcc dot gnu.org
@ 2014-05-29 9:23 ` amodra at gmail dot com
3 siblings, 0 replies; 5+ messages in thread
From: amodra at gmail dot com @ 2014-05-29 9:23 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61098
Alan Modra <amodra at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
URL|http://gcc.gnu.org/ml/gcc-p |https://gcc.gnu.org/ml/gcc-
|atches/2014-05/msg00451.htm |patches/2014-05/msg02024.ht
|l |ml
Resolution|--- |FIXED
Target Milestone|--- |4.10.0
--- Comment #3 from Alan Modra <amodra at gmail dot com> ---
Fixed mainline
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-29 9:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-07 14:59 [Bug target/61098] New: Poor code setting count register for large loops amodra at gmail dot com
2014-05-07 15:00 ` [Bug target/61098] " amodra at gmail dot com
2014-05-07 15:03 ` amodra at gmail dot com
2014-05-26 13:10 ` amodra at gcc dot gnu.org
2014-05-29 9:23 ` amodra at gmail dot com
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).