[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[rguenth at gcc dot gnu.org]

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

Richard Guenther  changed:

   What|Removed |Added

   Target Milestone|4.5.2   |4.5.3

--- Comment #52 from Richard Guenther  2010-12-16 
13:03:31 UTC ---
GCC 4.5.2 is being released, adjusting target milestone.

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[ubizjak at gmail dot com]

-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

   Severity|blocker |normal


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[ubizjak at gmail dot com]

--- Comment #51 from ubizjak at gmail dot com  2010-08-03 18:53 ---
Fixed, but there are certainly better ways to fix, see [1] and [2].

[1] http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00070.html
[2] http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00165.html

So, marked as suspended...


-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

 Status|NEW |SUSPENDED


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[uros at gcc dot gnu dot org]

--- Comment #50 from uros at gcc dot gnu dot org  2010-08-03 18:45 ---
Subject: Bug 41089

Author: uros
Date: Tue Aug  3 18:44:56 2010
New Revision: 162846

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162846
Log:
PR target/41089
* config/alpha/alpha.c (alpha_build_builtin_va_list): Mark __offset
as volatile.


Modified:
branches/gcc-4_5-branch/gcc/ChangeLog
branches/gcc-4_5-branch/gcc/config/alpha/alpha.c


-- 


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[ubizjak at gmail dot com]

--- Comment #49 from ubizjak at gmail dot com  2010-08-02 17:33 ---
Author: uros
Date: Mon Aug  2 17:26:40 2010
New Revision: 162826

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162826
Log:
target/41089
* config/alpha/alpha.c (alpha_build_builtin_va_list): Mark __offset
as volatile.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/alpha/alpha.c


-- 


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[ubizjak at gmail dot com]

--- Comment #48 from ubizjak at gmail dot com  2010-08-02 17:12 ---
Patch at http://gcc.gnu.org/ml/gcc-patches/2010-08/msg00021.html.


-- 

ubizjak at gmail dot com changed:

   What|Removed |Added

URL||http://gcc.gnu.org/ml/gcc-
   ||patches/2010-
   ||08/msg00021.html


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[ubizjak at gmail dot com]

--- Comment #47 from ubizjak at gmail dot com  2010-08-01 12:50 ---
(In reply to comment #39)

> or alpha could make the va_list struct copies volatile.  Or we can 
> schedule tree-stdarg earlier.

Well, following patch fixes remaining gcc.c-torture/execute/stdarg-1.c failure
as well:

Index: alpha/alpha.c
===
--- alpha/alpha.c   (revision 162794)
+++ alpha/alpha.c   (working copy)
@@ -5948,6 +5948,7 @@ alpha_build_builtin_va_list (void)
   ofs = build_decl (BUILTINS_LOCATION,
FIELD_DECL, get_identifier ("__offset"),
integer_type_node);
+  TREE_THIS_VOLATILE (ofs) = 1;
   DECL_FIELD_CONTEXT (ofs) = record;
   DECL_CHAIN (ofs) = space;


Does this patch makes sense w.r.t to stdarg optimizations?


-- 


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

[Bug tree-optimization/41089] [4.5/4.6 Regression] stdarg pass produces wrong code

[rguenth at gcc dot gnu dot org]

--- Comment #46 from rguenth at gcc dot gnu dot org  2010-07-31 09:29 
---
GCC 4.5.1 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Target Milestone|4.5.1   |4.5.2


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