This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Remove some typedefs (was: Silence class vs. struct warnings (opt_pass, ipa_opt_pass_d))
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Oleg Endo <oleg dot endo at t-online dot de>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 10 Dec 2013 15:50:12 +0100
- Subject: Re: Remove some typedefs (was: Silence class vs. struct warnings (opt_pass, ipa_opt_pass_d))
- Authentication-results: sourceware.org; auth=none
- References: <1386233288 dot 14008 dot 9 dot camel at yam-132-YW-E178-FTW> <3B985852-FCA6-44C3-9FEC-D18E3CEFAB68 at gmail dot com> <1386238371 dot 14008 dot 30 dot camel at yam-132-YW-E178-FTW> <CAFiYyc2ewWrBMR1OysL9zJrLpQbWLp+v15QeQxuhXQtmeEQXTg at mail dot gmail dot com> <1386243714 dot 14008 dot 38 dot camel at yam-132-YW-E178-FTW> <CAFiYyc2aHSXWDcBtEjRBb+H7TztnHnvYzNzGAVT3dBhkDGMUCw at mail dot gmail dot com> <1386254099 dot 14008 dot 43 dot camel at yam-132-YW-E178-FTW> <1386618599 dot 14008 dot 193 dot camel at yam-132-YW-E178-FTW>
On Mon, Dec 9, 2013 at 8:49 PM, Oleg Endo <oleg.endo@t-online.de> wrote:
> On Thu, 2013-12-05 at 15:34 +0100, Oleg Endo wrote:
>> On Thu, 2013-12-05 at 14:56 +0100, Richard Biener wrote:
>> >
>> > grep for 'typedef struct.*{' in headers. The typedef name is usually
>> > the desired one and is used without 'struct'. So it's an orthogonal
>> > issue.
>>
>> Ah, do you mean converting this stuff ...
>>
>> typedef struct
>> {
>> cgraph_node_set set;
>> unsigned index;
>> } cgraph_node_set_iterator;
>>
>> ... to ...
>>
>> struct cgraph_node_set_iterator
>> {
>> ....
>>
>> right?
>> Sure, no problem. But I'd rather do it step by step in separate
>> patches.
>
> So here it is.
> Tested with make all-gcc.
> Is this OK?
Ok.
Thanks,
Richard.
> Cheers,
> Oleg
>
> gcc/ChangeLog:
> * gcc/cgraph.h (cgraph_node_set_iterator,
> varpool_node_set_iterator): Remove typedef.
> (cgraph_inline_failed_enum, cgraph_inline_failed_t): Remove
> typedef and rename to cgraph_inline_failed_t.
> * gcc/tree-ssa-alias.h (ao_ref_s, ao_ref): Remove typedef and
> rename to ao_ref.
> * gcc/reload.h (reg_equivs_s, reg_equivs_t): Remove typedef and
> rename to reg_equivs_t.
> * gcc/conditions.h (CC_STATUS): Remove typedef.
> * gcc/bitmap.h (bitmap_obstack): Remove typedef.
> (bitmap_element_def, bitmap_element): Remove typedef and rename
> to bitmap_element.
> (bitmap_head_def, bitmap_head): Remove typedef and rename to
> bitmap_head.
> (bitmap_iterator): Remove typedef.
> * gcc/target.h (cumulative_args_t, print_switch_type,
> secondary_reload_info): Remove typedef.
> * gcc/dwarf2out.h (dw_cfi_oprnd_struct, dw_cfi_oprnd): Remove
> dw_cfi_oprnd_struct alias.
> (dw_cfi_struct, dw_cfi_node): Remove typedef and rename to
> dw_cfi_node.
> (dw_fde_struct, dw_fde_node): Remove typedef and rename to
> dw_fde_node.
> (cfa_loc, dw_cfa_location): Remove typedef and rename to
> dw_cfa_location.
> (dw_vec_struct, dw_vec_const): Remove typedef and rename to
> dw_vec_const.
> (dw_val_struct, dw_val_node): Remove typedef and rename to
> dw_val_node.
> (dw_loc_descr_struct, dw_loc_descr_node): Remove typedef and
> rename to dw_loc_descr_node.
> * gcc/params.h (param_info, compiler_param): Remove typedef.
> * gcc/opts.h (cl_deferred_param): Remove typedef.
> * gcc/sreal.h (sreal): Remove typedef.
> * gcc/ddg.h (dep_type, dep_data_type): Remove typedef.
> * gcc/graphite-clast-to-gimple.h (cloog_prog_clast, bb_pbb_def):
> Remove typedef.
> * gcc/lto-streamer.h (lto_decl_stream_e_t, lto_encoder_entry,
> lto_symtab_encoder_iterator, res_pair): Remove typedef.
> * gcc/tree-affine.h (affine_tree_combination, aff_tree): Remove
> typedef and rename to aff_tree.
> * gcc/sched-int.h (region): Remove typedef.
> * gcc/diagnostic.h (diagnostic_info,
> diagnostic_classification_change_t): Remove typedef.
> * gcc/tree-ssa-loop.h (affine_iv_d): Remove typedef and rename
> to affine_iv.
> * gcc/sbitmap.h (sbitmap_iterator): Remove typedef.
> * gcc/ssa-iterators.h (immediate_use_iterator_d,
> imm_use_iterator): Remove typedef and rename to
> imm_use_iterator.
> (ssa_operand_iterator_d, ssa_op_iter): Remove typedef and rename
> to ssa_op_iter.
> * gcc/ggc-internal.h (ggc_statistics): Remove typedef.
> * gcc/cselib.h (cselib_val_struct, cselib_val): Remove typedef
> and rename to cselib_val.
> * gcc/tree-core.h (alias_pair): Remove typedef.
> (constructor_elt_d, constructor_elt): Remove typedef and rename
> to constructor_elt.
> (ssa_use_operand_d, ssa_use_operand_t): Remove typedef and
> rename to ssa_use_operand_t.
> * gcc/graphite-sese-to-poly.h (base_alias_pair): Remove typedef.
> * gcc/tree-data-ref.h (conflict_function): Remove typedef.
> * gcc/tree-inline.h (copy_body_data): Remove typedef.
> * gcc/ipa-inline.h (condition, size_time_entry,
> inline_param_summary_t, edge_growth_cache_entry): Remove
> typedef.
> * gcc/regrename.h (operand_rr_info, insn_rr_info): Remove
> typedef.
> * gcc/gimple-iterator.h (gimple_stmt_iterator_d,
> gimple_stmt_iterator): Remove typedef and rename to
> gimple_stmt_iterator.
> * gcc/basic-block.h (ce_if_block, ce_if_block_t): Remove typedef
> and rename to ce_if_block.
> (edge_iterator): Remove typedef.
> * gcc/ipa-prop.h (ipa_agg_jf_item, ipa_agg_jf_item_t): Remove
> typedef and rename to ipa_agg_jf_item.
> (ipa_agg_jump_function_t, ipa_param_descriptor_t,
> ipa_node_params_t, ipa_parm_adjustment_t): Remove typedef.
> (ipa_jump_func, ipa_jump_func_t): Remove typedef and rename to
> ipa_jump_func.
> (ipa_edge_args, ipa_edge_args_t): Remove typedef and rename to
> ipa_edge_args.
> * gcc/gcov-io.h (gcov_bucket_type): Remove typedef.
> (gcov_working_set_info, gcov_working_set_t): Remove typedef and
> rename to gcov_working_set_t.
> * gcc/ira-int.h (minmax_set_iterator, ira_allocno_iterator,
> ira_object_iterator, ira_allocno_object_iterator,
> ira_pref_iterator, ira_copy_iterator,
> ira_object_conflict_iterator): Remove typedef.
> * gcc/tree-iterator.h (tree_stmt_iterator): Remove typedef.
> * gcc/rtl.h (addr_diff_vec_flags, mem_attrs, reg_attrs,
> replace_label_data): Remove typedef.
> (rtunion_def, rtunion): Remove typedef and rename to rtunion.
> * gcc/hard-reg-set.h (hard_reg_set_iterator): Remove typedef.
> * gcc/sel-sched-ir.h (_list_iterator, sel_global_bb_info_def,
> sel_region_bb_info_def, succ_iterator): Remove typedef.
> (deps_where_def, deps_where_t): Remove typedef and rename to
> deps_where_t.
>
> * gcc/coretypes.h: Adapt forward declarations.
> * gcc/tree-scalar-evolution.h: Likewise.
> * gcc/tree-ssa-address.h: Likewise.
> * gcc/tree-ssa-operands.h: Likewise.
> * gcc/function.h: Likewise.
> * gcc/config/frv/frv-protos.h: Likewise.
> * gcc/targhooks.h: Likewise.
> * gcc/basic_block.h: Likewise.
>
> * gcc/rtl.def: Adapt documentation.
> * gcc/doc/tm.texi: Likewise.
>
> * gcc/ipa-cp.c: Adapt uses.
> * gcc/bitmap.c: Likewise.
> * gcc/dwarf2out.c: Likewise.
> * gcc/target.def: Likewise.
> * gcc/ipa-inline-analysis.c: Likewise.
> * gcc/dwarf2cfi.c: Likewise.
> * gcc/tree-ssa-loop-ivopts.c: Likewise.
> * gcc/lto-cgraph.c: Likewise.
> * gcc/config/frv/frv.c: Likewise.
> * gcc/ifcvt.c: Likewise.
> * gcc/ipa-prop.c: Likewise.
>