From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 83851 invoked by alias); 9 Oct 2018 20:34:36 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 83840 invoked by uid 89); 9 Oct 2018 20:34:35 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_PASS autolearn=ham version=3.3.2 spammy=H*UA:https X-HELO: mx1.suse.de Received: from mx2.suse.de (HELO mx1.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 09 Oct 2018 20:34:33 +0000 X-Amavis-Alert: BAD HEADER SECTION, Duplicate header field: "Cc" Received: from relay1.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 16010ADDF; Tue, 9 Oct 2018 20:34:31 +0000 (UTC) From: Martin Jambor To: Martin =?utf-8?Q?Li=C5=A1ka?= , Bernhard Reutner-Fischer , Jeff Law Cc: Richard Guenther , GCC Patches Cc: Subject: Re: [PATCH] Remove dead functions and fix VMS target by moving back some functions. In-Reply-To: <794c78c4-0950-420c-2fd4-21b5e17d881f@suse.cz> References: <192f112275eb2c7bc74fdd3b91c0b32882695d22.1537774329.git.mliska@suse.cz> <43d7b949-b909-6c5a-866a-d7a8f56e26b2@redhat.com> <5386add0-0af1-d604-96e6-dab0e565924b@suse.cz> <139d57c2-c70b-3852-0e20-8d19e6ae6bce@suse.cz> <20180927131337.1104d328@nbbrfq.loc> <3cc447f0-5cf3-3c0b-4293-f64059605871@suse.cz> <794c78c4-0950-420c-2fd4-21b5e17d881f@suse.cz> User-Agent: Notmuch/0.26 (https://notmuchmail.org) Emacs/26.1 (x86_64-suse-linux-gnu) Date: Tue, 09 Oct 2018 22:53:00 -0000 Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2018-10/txt/msg00527.txt.bz2 On Tue, Oct 09 2018, Martin Li=C5=A1ka wrote: > Hi. > > Utilizing rtags' --find-dead-functions I'm suggesting a removal of part > of the functions reported with the script. I built all cross compilers > defined in contrib/config-list.mk and I fixed VMS targets that I broke > in previous removal. > > If the folks are happy with the removal, I can probably continue with that > at some point in stage3 or so. > > Ready for trunk? > Thanks, > Martin > From f69d866da42a46783ab57181692583d2ecf30c49 Mon Sep 17 00:00:00 2001 > From: marxin > Date: Fri, 5 Oct 2018 16:59:07 +0200 > Subject: [PATCH] Remove dead functions and fix VMS target by moving back = some > functions. > > gcc/ChangeLog: > > 2018-10-09 Martin Liska > > * attribs.c (register_attribute): Remove unused functions. > * cfganal.c (control_dependences::clear_control_dependence_bitmap): Like= wise. > (bitmap_union_of_succs): Likewise. > * cfganal.h (bitmap_union_of_succs): Likewise. > * cfgloop.c (loop_exits_to_bb_p): Likewise. > * cfgloop.h (loop_exits_to_bb_p): Likewise. > * cgraph.h (compute_call_stmt_bb_frequency): Likewise. > * cgraphbuild.c (compute_call_stmt_bb_frequency): Likewise. > * cselib.c (cselib_dummy_expand_value_rtx_cb): Likewise. > * cselib.h (cselib_dummy_expand_value_rtx_cb): Likewise. > * df-core.c (df_reg_defined): Likewise. > (df_reg_used): Likewise. > * df-scan.c (df_hard_reg_used_count): Likewise. > * df.h (df_reg_defined): Likewise. > (df_reg_used): Likewise. > (df_hard_reg_used_count): Likewise. > * dojump.c (init_pending_stack_adjust): Likewise. > * dojump.h (init_pending_stack_adjust): Likewise. > * dwarf2out.c (add_AT_vms_delta): Move function back after > removal. > (dwarf2out_set_demangle_name_func): REmove unused functions. > * dwarf2out.h (dwarf2out_set_demangle_name_func): Likewise. > * emit-rtl.c (emit_jump_insn_before_noloc): Likewise. > (emit_call_insn_before_noloc): Likewise. > (emit_barrier_before): Likewise. > (emit_call_insn_after_noloc): Likewise. > (emit_call_insn_after_setloc): Likewise. > (emit_jump_insn_before_setloc): Likewise. > (emit_call_insn_before_setloc): Likewise. > (emit_call_insn_before): Likewise. > * except.c (remove_eh_handler): Likewise. > (get_eh_region_from_rtx): Likewise. > * except.h (remove_eh_handler): Likewise. > (get_eh_region_from_rtx): Likewise. > * fold-const.c (div_if_zero_remainder): Likewise. > (fold_unary_to_constant): Likewise. > * fold-const.h (fold_unary_to_constant): Likewise. > (div_if_zero_remainder): Likewise. > * function.c (get_hard_reg_initial_reg): Likewise. > (get_last_funcdef_no): Likewise. > * function.h (get_hard_reg_initial_reg): Likewise. > (get_last_funcdef_no): Likewise. > * ggc-common.c (ggc_cleared_alloc_htab_ignore_args): Likewise. > (ggc_cleared_alloc_ptr_array_two_args): Likewise. > (ggc_splay_alloc): Likewise. > (ggc_splay_dont_free): Likewise. > * gimple-expr.c (is_gimple_address): Likewise. > * gimple-expr.h (is_gimple_address): Likewise. > * gimple-iterator.c (gsi_for_phi): Likewise. > * gimple-iterator.h (gsi_for_phi): Likewise. > * gimple-ssa-sprintf.c (struct format_result): Likewise. > (format_result::operator+=3D): Likewise. > (struct directive): Likewise. > * gimple-ssa-warn-alloca.c (struct alloca_type_and_limit): Likewise. > * gimple.c (gimple_unsigned_type): Likewise. > (gimple_call_builtin_p): Likewise. > * gimple.h (gimple_unsigned_type): Likewise. > (gimple_call_builtin_p): Likewise. > * gimplify.c (gimple_current_bind_expr): Likewise. > * gimplify.h (gimple_current_bind_expr): Likewise. > * haifa-sched.c (haifa_classify_insn): Likewise. > * hsa-common.c (hsa_add_kernel_dependency): Likewise. > (hsa_brig_function_name): Likewise. > * hsa-common.h (hsa_add_kernel_dependency): Likewise. > (hsa_brig_function_name): Likewise. > * ipa-cp.c (ipcp_vr_lattice::meet_with): Likewise. Interesting, I believe that a class representing a lattice should have a meet function with its own type. In this case it would mean moving the call to ipa_vr_operation_and_type_effects from the caller into the method to clearly describe the semantics. But perhaps it does not matter if we do not have the overload in the meantime. > * ipa-devirt.c (vtable_pointer_value_to_binfo): Likewise. > * ipa-icf-gimple.c (func_checker::compare_tree_ssa_label): Likewise. > * ipa-icf-gimple.h: Likewise. > * ipa-icf.c (sem_item::sem_item): Likewise. > (sem_function::sem_function): Likewise. > (sem_function::icf_handled_component_p): Likewise. > (sem_variable::sem_variable): Likewise. > * ipa-icf.h: Likewise. > * ipa-param-manipulation.c (index_in_adjustments_multiple_times_p): Like= wise. > (ipa_combine_adjustments): Likewise. This a known case. I mean, also have a patch removing, actually replacing it, but the goal has always been that we eventually combine adjustments. Martin