> -----Original Message-----
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On
> Behalf Of Richard Biener
> Sent: 15 October 2013 14:34
> To: Paulo Matos
> Cc: Jakub Jelinek; Paolo Carlini; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] tree_code_name wrapper
> 
> On Tue, Oct 15, 2013 at 3:19 PM, Paulo Matos <pma...@broadcom.com> wrote:
> >> -----Original Message-----
> >> From: Jakub Jelinek [mailto:ja...@redhat.com]
> >> Sent: 15 October 2013 10:51
> >> To: Paulo Matos
> >> Cc: Richard Biener; Paolo Carlini; gcc-patches@gcc.gnu.org
> >> Subject: Re: [PATCH] tree_code_name wrapper
> >>
> >> On Tue, Oct 15, 2013 at 09:42:17AM +0000, Paulo Matos wrote:
> >> > Thanks, regarding the indentation I was convinced we used 2 space
> >> > indentation with maximum line length of 80 characters.
> >>
> >> We do, however 8 consecutive spaces in the indentation should be always
> >> replaced by a tab.
> >>
> >
> > This means that every sequence of 8 spaces should be converted into tabs?
> So, if we indent something 4 times, that becomes a tab instead of 4 times 2
> spaces.
> 
> Correct.
> 
> Richard.
> 

Thanks. Updated patch attached. Ok to submit?

2013-10-15  Paulo Matos  <pma...@broadcom.com>

gcc/
        * tree-core.h: Remove extern declaration of tree_code_name.
        * tree.h: Prototype for new function get_tree_code_name.
        * tree.c: Make tree_code_name static. Define new function
        wrapper get_tree_code_name.
        (dump_tree_statistics, tree_check_failed, tree_not_check_failed, 
        tree_class_check_failed, tree_range_check_failed, 
tree_not_class_check_failed,
        omp_clause_check_failed, tree_contains_struct_check_failed, 
        tree_operand_check_failed):      Use new wrapper get_tree_code_name 
instead of
        calling tree_code_name directly.
        * tree-vrp.c (dump_asserts_for): Likewise.
        * tree-dump.c (dequeue_and_dump): Likewise.
        * tree-pretty-print.c (do_niy, dump_generic_node): Likewise.
        * tree-pretty-print.h (pp_unsupported_tree): Likewise.
        * lto-streamer-out.c (lto_write_tree, DFS_write_tree): Likewise.
        * tree-ssa-dom.c (print_expr_hash_elt): Likewise.
        * gimple-pretty-print.c (dump_unary_rhs, dump_binary_rhs, 
dump_ternary_rhs, 
        dump_gimple_assign, dump_gimple_cond, dump_gimple_omp_for): Likewise.
        * tree-vect-data-refs.c (vect_create_data_ref_ptr): Likewise.
        * tree-ssa-pre.c (print_pre_expr): Likewise.
        * ipa-prop.c (ipa_print_node_jump_functions_for_edge): Likewise.
        * print-tree.c (print_node_brief, print_node): Likewise.
        * gimple.c (gimple_check_failed): Likewise.
        * lto-streamer.c (lto_tag_name, print_lto_report): Likewise.

gcc/cp/
        * error.c (code_to_string): Use new wrapper get_tree_code_name.
        * cxx-pretty-print.c (pp_cxx_assignment_operator): Likewise.
        * pt.c (tsubst): Likewise.
        * semantics.c (cxx_eval_constant_expression, 
potential_constant_expression_1): Likewise.
        * mangle.c (MANGLE_TRACE_TREE, dump_substitution_candidates, 
add_substitution,
        find_substitution): Likewise.

gcc/config/
        * frv/frv.c (frv_init_cumulative_args): Use new wrapper 
get_tree_code_name.
        * mep/mep.c (mep_validate_vliw): Likewise.
        * iq2000/iq2000.c (init_cumulative_args): Likewise.
        * rs6000/rs6000.c (init_cumulative_args): Likewise.

Attachment: tree_code_name-v2.patch
Description: tree_code_name-v2.patch

Reply via email to