http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
--- Comment #18 from Ryan Hill 2010-12-26
08:03:51 UTC ---
Ignore that.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
Ryan Hill changed:
What|Removed |Added
CC||dirtyepic at gentoo dot org
--- Comment #17 f
--- Comment #16 from mikpe at it dot uu dot se 2010-09-20 16:37 ---
FWIW, exposed on trunk by r160462 (PR44423 fix), backported to 4.5 in r160775.
But clearly the issue was latent since the movt patterns were added.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
--- Comment #15 from rearnsha at gcc dot gnu dot org 2010-09-20 16:22
---
Subject: Bug 45726
Author: rearnsha
Date: Mon Sep 20 16:21:57 2010
New Revision: 164441
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164441
Log:
2010-09-20 Rafael Carre
PR target/45726
--- Comment #14 from rearnsha at gcc dot gnu dot org 2010-09-20 16:13
---
(In reply to comment #13)
> Is there something wrong with the first hunk of the patch (arm_movt) ?
>
Nothing really. I missed that bit.
I think in practice the compiler will never end up matching that pattern
--- Comment #13 from rafael dot carre at gmail dot com 2010-09-20 15:46
---
Is there something wrong with the first hunk of the patch (arm_movt) ?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
--- Comment #12 from rearnsha at gcc dot gnu dot org 2010-09-20 15:36
---
Fixed in 4.5.3 and trunk.
--
rearnsha at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from rearnsha at gcc dot gnu dot org 2010-09-20 15:27
---
Subject: Bug 45726
Author: rearnsha
Date: Mon Sep 20 15:27:13 2010
New Revision: 164437
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164437
Log:
2010-09-20 Rafael Carre
PR target/45726
--- Comment #10 from rearnsha at gcc dot gnu dot org 2010-09-20 15:25
---
Subject: Bug 45726
Author: rearnsha
Date: Mon Sep 20 15:25:44 2010
New Revision: 164436
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164436
Log:
2010-09-20 Rafael Carre
PR target/45726
--
rearnsha at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |rearnsha at gcc dot gnu dot
|dot org
--- Comment #9 from rearnsha at gcc dot gnu dot org 2010-09-20 15:20
---
Must also be present (even if latent) on 4.5.
--
rearnsha at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from mikpe at it dot uu dot se 2010-09-20 12:02 ---
r139881 is good. I'll start a bisection.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
--- Comment #7 from rafael dot carre at gmail dot com 2010-09-20 11:08
---
I didn't bisect.
Did you try r139881? If I download a checkout (I'm using a snapshot) to bisect
I could use this as a working starting point.
TARGET_USE_MOVT uses arm_arch_thumb2 just like my patch so it's not b
--- Comment #6 from mikpe at it dot uu dot se 2010-09-20 10:29 ---
Can you do a bisection to identify the exact commit responsible? Looking at
the original commit that introduced the movt md pattern (139881) I see a
TARGET_USE_MOVT guard in the C code that _should_ prevent it from being
--- Comment #5 from rafael dot carre at gmail dot com 2010-09-20 08:17
---
Created an attachment (id=21845)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21845&action=view)
Only emit MOVT when the targetted CPU is Thumb2-able
--
rafael dot carre at gmail dot com changed:
--- Comment #4 from rafael dot carre at gmail dot com 2010-09-20 07:47
---
Created an attachment (id=21844)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21844&action=view)
Only emit MOVT when targetting Thumb2
Tentative patch.
However as suggested by the original TARGET_32BIT,
--- Comment #3 from rafael dot carre at gmail dot com 2010-09-20 07:24
---
I made a CVS checkout of binutils yesterday.
You're right, MOVT is supported on ARMv7 because all ARMv7 supports Thumb2,
http://infocenter.arm.com/help/topic/com.arm.doc.qrc0001m/QRC0001_UAL.pdf
(It says "All Th
--- Comment #2 from pinskia at gcc dot gnu dot org 2010-09-20 04:52 ---
What binutils version are you using?
movteq is a valid ARM v7 instruction.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45726
18 matches
Mail list logo
