[Bug tree-optimization/27532] __builtin_object_size does not handle a + b.

2006-05-29 Thread jakub at gcc dot gnu dot org 


--- Comment #6 from jakub at gcc dot gnu dot org  2006-05-29 09:54 ---
Subject: Bug 27532

Author: jakub
Date: Mon May 29 09:54:11 2006
New Revision: 114193

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=114193
Log:
2006-05-10  Richard Guenther  [EMAIL PROTECTED]

PR tree-optimization/27532
* tree-object-size.c (plus_expr_object_size): Fix typo.

Modified:
branches/redhat/gcc-4_1-branch/gcc/ChangeLog
branches/redhat/gcc-4_1-branch/gcc/tree-object-size.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=27532

[Bug tree-optimization/27532] __builtin_object_size does not handle a + b.

2006-05-10 Thread rguenth at gcc dot gnu dot org 


--- Comment #2 from rguenth at gcc dot gnu dot org  2006-05-10 14:48 ---
Looks more like a typo in tree-object-size.c:plus_expr_object_size

Index: tree-object-size.c
===
*** tree-object-size.c  (revision 113669)
--- tree-object-size.c  (working copy)
*** plus_expr_object_size (struct object_siz
*** 595,601 
{
  unsigned HOST_WIDE_INT off = tree_low_cst (op1, 1);

! bytes = compute_builtin_object_size (value, object_size_type);
  if (off  offset_limit)
bytes = unknown[object_size_type];
  else if (off  bytes)
--- 595,601 
{
  unsigned HOST_WIDE_INT off = tree_low_cst (op1, 1);

! bytes = compute_builtin_object_size (op0, object_size_type);
  if (off  offset_limit)
bytes = unknown[object_size_type];
  else if (off  bytes)

mine.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 CC|dnovillo at redhat dot com  |
 AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
   |dot org |org
 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2006-05-10 14:48:34
   date||
Summary|CCP produces non-gimple |__builtin_object_size does
   |statements  |not handle a + b.


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=27532

[Bug tree-optimization/27532] __builtin_object_size does not handle a + b.

2006-05-10 Thread pinskia at gcc dot gnu dot org 


--- Comment #3 from pinskia at gcc dot gnu dot org  2006-05-10 15:10 ---
Just to explain why a[0] + 2B is valid gimple:
a[0] is a gimple invariant and likewise for 2b. Therefor the two sides of
the Plus expression are going to valid for it.  Therefor valid gimple.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=27532

[Bug tree-optimization/27532] __builtin_object_size does not handle a + b.

2006-05-10 Thread rguenth at gcc dot gnu dot org 


--- Comment #4 from rguenth at gcc dot gnu dot org  2006-05-10 19:41 ---
Subject: Bug 27532

Author: rguenth
Date: Wed May 10 19:41:46 2006
New Revision: 113682

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=113682
Log:
2006-05-10  Richard Guenther  [EMAIL PROTECTED]

PR tree-optimization/27532
* tree-object-size.c (plus_expr_object_size): Fix typo.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-object-size.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=27532

[Bug tree-optimization/27532] __builtin_object_size does not handle a + b.

2006-05-10 Thread rguenth at gcc dot gnu dot org 


--- Comment #5 from rguenth at gcc dot gnu dot org  2006-05-10 19:42 ---
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.2.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=27532