[Bug target/46788] unsigned int possible treated as signed in a union/struct

2012-01-12 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

Andrew Pinski  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED

--- Comment #11 from Andrew Pinski  2012-01-12 
20:29:20 UTC ---
Fixed.


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-20 Thread mikpe at it dot uu.se
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #10 from Mikael Pettersson  2011-03-20 
16:40:14 UTC ---
(In reply to comment #8)
> Author: ramana
> Date: Tue Mar 15 19:59:25 2011
> New Revision: 171017
> 
> URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171017
> Log:
> 
> Fix PR target/46788
> 
> 
> Added:
> branches/gcc-4_6-branch/gcc/testsuite/gcc.target/arm/pr46788.c
>   - copied unchanged from r171005,
> trunk/gcc/testsuite/gcc.target/arm/pr46788.c
> Modified:
> branches/gcc-4_6-branch/gcc/ChangeLog
> branches/gcc-4_6-branch/gcc/config/arm/arm.md
> branches/gcc-4_6-branch/gcc/testsuite/ChangeLog

The ChangeLog entry refers to PR 46778, it should refer to PR 46788.


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-15 Thread ramana at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #9 from Ramana Radhakrishnan  2011-03-15 
20:06:16 UTC ---
(In reply to comment #7)
> AFAICS, however, pr47688.c is still there.

I think I managed to delete it now even though I did do a svn rename.

Ramana


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-15 Thread ramana at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #8 from Ramana Radhakrishnan  2011-03-15 
19:59:29 UTC ---
Author: ramana
Date: Tue Mar 15 19:59:25 2011
New Revision: 171017

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171017
Log:

Fix PR target/46788


Added:
branches/gcc-4_6-branch/gcc/testsuite/gcc.target/arm/pr46788.c
  - copied unchanged from r171005,
trunk/gcc/testsuite/gcc.target/arm/pr46788.c
Modified:
branches/gcc-4_6-branch/gcc/ChangeLog
branches/gcc-4_6-branch/gcc/config/arm/arm.md
branches/gcc-4_6-branch/gcc/testsuite/ChangeLog


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-15 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #7 from Paolo Carlini  2011-03-15 
17:16:11 UTC ---
AFAICS, however, pr47688.c is still there.


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-15 Thread ramana at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #6 from Ramana Radhakrishnan  2011-03-15 
17:07:56 UTC ---

(In reply to comment #5)
> Author: ramana
> Date: Tue Mar 15 17:05:51 2011
> New Revision: 171002
> 
> URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171002
> Log:
> 
> Fixup last commit.
> 
> Fixed PR target/46788 and not PR 47688
> 
> 
> Added:
> trunk/gcc/testsuite/gcc.target/arm/pr46788.c
>   - copied unchanged from r171001,
> trunk/gcc/testsuite/gcc.target/arm/pr47688.c
> Modified:
> trunk/gcc/ChangeLog
> trunk/gcc/testsuite/ChangeLog


This was fixed on trunk with this original commit followed by the commit in the
previous comment.


Author: ramana
Date: Tue Mar 15 16:14:21 2011
New Revision: 171000

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171000
Log:
Fix PR 47688

2011-03-18  Ramana Radhakrishnan  

PR target/47668
gcc/
* config/arm/arm.md (arm_movtas_ze): Use 'L' instead of 'c'
in the output template.
gcc/testsuite/
* gcc.target/arm/pr47688.c: New.

Added:
trunk/gcc/testsuite/gcc.target/arm/pr47688.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/arm/arm.md
trunk/gcc/testsuite/ChangeLog


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-15 Thread ramana at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

--- Comment #5 from Ramana Radhakrishnan  2011-03-15 
17:05:56 UTC ---
Author: ramana
Date: Tue Mar 15 17:05:51 2011
New Revision: 171002

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171002
Log:

Fixup last commit.

Fixed PR target/46788 and not PR 47688


Added:
trunk/gcc/testsuite/gcc.target/arm/pr46788.c
  - copied unchanged from r171001,
trunk/gcc/testsuite/gcc.target/arm/pr47688.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-03-10 Thread ramana at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

Ramana Radhakrishnan  changed:

   What|Removed |Added

   Keywords||wrong-code
 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2011.03.10 11:59:32
   Target Milestone|--- |4.5.3
 Ever Confirmed|0   |1

--- Comment #4 from Ramana Radhakrishnan  2011-03-10 
11:59:32 UTC ---
Confirmed


[Bug target/46788] unsigned int possible treated as signed in a union/struct

2011-02-04 Thread mikpe at it dot uu.se
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46788

Mikael Pettersson  changed:

   What|Removed |Added

 CC||ramana at gcc dot gnu.org

--- Comment #3 from Mikael Pettersson  2011-02-04 
10:35:43 UTC ---
It's caused by 150953:

Author: ramana
Date: Thu Aug 20 08:09:29 2009
New Revision: 150953

URL: http://gcc.gnu.org/viewcvs?root=3Dgcc&view=3Drev&rev=3D150953
Log:
Fix twolf ICE for ARM

2009-08-19  Ramana Radhakrishnan  
Richard Earnshaw  

* config/arm/arm.c (arm_emit_movpair): Handle CONST_INT.
* config/arm/arm.md (*arm_movtas_ze): New pattern for
movt.

You're missing an "#:upper16:" annotation in the *arm_movtas_ze pattern, c.f.
the *arm_movt pattern.

(Not posting a patch since my renewed attempt to get a copyright assignment
seems to have ended up in /dev/null over at the FSF.)