From patchwork Tue Jun 22 18:30:15 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: trunk patch: moving gimple_seq_node from coretypes.h to gimple.h Date: Tue, 22 Jun 2010 08:30:15 -0000 From: Basile Starynkevitch X-Patchwork-Id: 56562 Message-Id: <1277231415.19557.20.camel@glinka> To: gcc-patches@gcc.gnu.org Cc: "Joseph S.Myers" Hello All, since gimple_seq_node is not very useful in coretypes.h I bootstrapped the following attached patch to trunk rev.161214 on x86_64-unknown-linux-gnu with c,c++,lto languages. gcc/ChangeLog entry: 2010-06-22 Basile Starynkevitch * coretypes.h (gimple_seq_node_d, gimple_seq_node) (const_gimple_seq_node): Removed typedefs. * gimple.h (gimple_seq_node_d, gimple_seq_node) (const_gimple_seq_node): Added typedefs moved from coretypes.h. See also http://gcc.gnu.org/ml/gcc/2010-06/msg00670.html & http://gcc.gnu.org/ml/gcc/2010-06/msg00657.html By the way, I would also happy if someone could also review the GNU-friendly gengtype patch http://gcc.gnu.org/ml/gcc-patches/2010-06/msg02178.html Cheers Index: gcc/coretypes.h =================================================================== --- gcc/coretypes.h (revision 161214) +++ gcc/coretypes.h (working copy) @@ -68,9 +68,6 @@ struct cl_optimization; struct gimple_seq_d; typedef struct gimple_seq_d *gimple_seq; typedef const struct gimple_seq_d *const_gimple_seq; -struct gimple_seq_node_d; -typedef struct gimple_seq_node_d *gimple_seq_node; -typedef const struct gimple_seq_node_d *const_gimple_seq_node; /* Address space number for named address space support. */ typedef unsigned char addr_space_t; Index: gcc/gimple.h =================================================================== --- gcc/gimple.h (revision 161214) +++ gcc/gimple.h (working copy) @@ -33,6 +33,12 @@ along with GCC; see the file COPYING3. If not see #include "tree-ssa-operands.h" #include "tree-ssa-alias.h" +/* The types gimple & gimple_seq are defined in coretypes.h but + gimple_seq_node is not needed there. */ +struct gimple_seq_node_d; +typedef struct gimple_seq_node_d *gimple_seq_node; +typedef const struct gimple_seq_node_d *const_gimple_seq_node; + /* For each block, the PHI nodes that need to be rewritten are stored into these vectors. */ typedef VEC(gimple, heap) *gimple_vec;