On 09/23/2015 10:05 AM, Kyrill Tkachov wrote:
Hi all,I notice that the functions count_insns_seq and estimate_num_insns_seq perform the exact same function for exactly the same arguments. It's redundant to keep both around. I've decided to delete count_insns_seq and replace its one use by estimate_num_insns_seq. Bootstrapped and tested on aarch64, x86_64. I think this change is obvious, so I'll commit it in 24 hours unless someone objects. Thanks, Kyrill 2015-09-23 Kyrylo Tkachov <kyrylo.tkac...@arm.com> * tree-inline.h (count_insns_seq): Delete prototype. (estimate_num_insns_seq): Define prototype. * tree-inline.c (count_insns_seq): Delete. (estimate_num_insns_seq): Remove static qualifier. * tree-eh.c (decide_copy_try_finally): Replace use of count_insns_seq with estimate_num_insns_seq.
This is fine and I think would fall under the "obvious" rule. Note that we don't have a "no objections" rule. Jeff
