* [Bug target/39716] [4.5 Regression][cond-optab] worse MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
@ 2009-05-12 12:06 ` bonzini at gnu dot org
2009-05-21 10:37 ` rguenth at gcc dot gnu dot org
` (6 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: bonzini at gnu dot org @ 2009-05-12 12:06 UTC (permalink / raw)
To: gcc-bugs
--
bonzini at gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.5.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] [4.5 Regression][cond-optab] worse MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
2009-05-12 12:06 ` [Bug target/39716] [4.5 Regression][cond-optab] " bonzini at gnu dot org
@ 2009-05-21 10:37 ` rguenth at gcc dot gnu dot org
2009-07-14 20:35 ` bonzini at gnu dot org
` (5 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2009-05-21 10:37 UTC (permalink / raw)
To: gcc-bugs
--
rguenth at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
GCC target triplet| |arm
Priority|P3 |P2
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] [4.5 Regression][cond-optab] worse MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
2009-05-12 12:06 ` [Bug target/39716] [4.5 Regression][cond-optab] " bonzini at gnu dot org
2009-05-21 10:37 ` rguenth at gcc dot gnu dot org
@ 2009-07-14 20:35 ` bonzini at gnu dot org
2009-08-19 8:59 ` ramana at gcc dot gnu dot org
` (4 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: bonzini at gnu dot org @ 2009-07-14 20:35 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from bonzini at gnu dot org 2009-07-14 20:35 -------
This fails for me with r149508 with a reload failure.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] [4.5 Regression][cond-optab] worse MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
` (2 preceding siblings ...)
2009-07-14 20:35 ` bonzini at gnu dot org
@ 2009-08-19 8:59 ` ramana at gcc dot gnu dot org
2009-09-09 11:08 ` ramana at gcc dot gnu dot org
` (3 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ramana at gcc dot gnu dot org @ 2009-08-19 8:59 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from ramana at gcc dot gnu dot org 2009-08-19 08:59 -------
(In reply to comment #1)
> This fails for me with r149508 with a reload failure.
>
I don't seem to have a reload failure for this with 150624.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] [4.5 Regression][cond-optab] worse MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
` (3 preceding siblings ...)
2009-08-19 8:59 ` ramana at gcc dot gnu dot org
@ 2009-09-09 11:08 ` ramana at gcc dot gnu dot org
2009-09-09 15:11 ` [Bug target/39716] suboptimal " bonzini at gnu dot org
` (2 subsequent siblings)
7 siblings, 0 replies; 9+ messages in thread
From: ramana at gcc dot gnu dot org @ 2009-09-09 11:08 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from ramana at gcc dot gnu dot org 2009-09-09 11:08 -------
There isn't a reload failure with trunk as of yesterday.
Paolo can you explain this further ?
--
ramana at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |WAITING
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] suboptimal MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
` (4 preceding siblings ...)
2009-09-09 11:08 ` ramana at gcc dot gnu dot org
@ 2009-09-09 15:11 ` bonzini at gnu dot org
2010-02-08 11:29 ` steven at gcc dot gnu dot org
2010-02-08 11:54 ` rguenth at gcc dot gnu dot org
7 siblings, 0 replies; 9+ messages in thread
From: bonzini at gnu dot org @ 2009-09-09 15:11 UTC (permalink / raw)
To: gcc-bugs
------- Comment #4 from bonzini at gnu dot org 2009-09-09 15:11 -------
I don't see the reload failure anymore.
The code is:
push {r4, lr}
mov r4, r0
mov r1, #0
bl __gesf2
cmp r0, #0
it lt
movlt r4, #0
mov r0, r4
pop {r4, pc}
Pre-cond-optab, we generated
push {r4, lr}
mov r4, r0
mov r1, #0
bl __gesf2
cmp r0, #0
mov r0, #0 <<<<
it ge
movge r0, r4
pop {r4, pc}
So this is fixed. I'm leaving this bug open as an enhancement though because
the highlighted mov is redundant.
--
bonzini at gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
OtherBugsDependingO|39714 |
nThis| |
Severity|normal |enhancement
Status|WAITING |NEW
Ever Confirmed|0 |1
Keywords| |missed-optimization
Priority|P2 |P3
Last reconfirmed|0000-00-00 00:00:00 |2009-09-09 15:11:09
date| |
Summary|[4.5 Regression][cond-optab]|suboptimal MAX_EXPR
|worse MAX_EXPR expansion for|expansion for Thumb
|Thumb |
Target Milestone|4.5.0 |---
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] suboptimal MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
` (5 preceding siblings ...)
2009-09-09 15:11 ` [Bug target/39716] suboptimal " bonzini at gnu dot org
@ 2010-02-08 11:29 ` steven at gcc dot gnu dot org
2010-02-08 11:54 ` rguenth at gcc dot gnu dot org
7 siblings, 0 replies; 9+ messages in thread
From: steven at gcc dot gnu dot org @ 2010-02-08 11:29 UTC (permalink / raw)
To: gcc-bugs
------- Comment #5 from steven at gcc dot gnu dot org 2010-02-08 11:29 -------
Trunk today (r156595) produces this:
repl1:
@ args = 0, pretend = 0, frame = 0
@ frame_needed = 0, uses_anonymous_args = 0
push {r4, lr}
mov r4, r0
mov r1, #0
bl __gesf2
cmp r0, #0
it lt
movlt r4, #0
mov r0, r4
pop {r4, pc}
Regarding the redundant mov of comment #4: There is another PR about missed
implicit sets after reload, see bug 14770. I'm sure this is the same issue.
Thus, fixed.
--
steven at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Bug target/39716] suboptimal MAX_EXPR expansion for Thumb
2009-04-10 12:44 [Bug target/39716] New: [cond-optab] worse MAX_EXPR expansion for Thumb bonzini at gnu dot org
` (6 preceding siblings ...)
2010-02-08 11:29 ` steven at gcc dot gnu dot org
@ 2010-02-08 11:54 ` rguenth at gcc dot gnu dot org
7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2010-02-08 11:54 UTC (permalink / raw)
To: gcc-bugs
--
rguenth at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.5.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39716
^ permalink raw reply [flat|nested] 9+ messages in thread