[PATCH][tree-inline][obvious] Delete redundant count_insns_seq

Kyrill Tkachov kyrylo.tkachov@arm.com
Wed Sep 23 16:18:00 GMT 2015


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.tkachov@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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: delete-count_insns_seq.patch
Type: text/x-patch
Size: 2318 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20150923/5546fc66/attachment.bin>


More information about the Gcc-patches mailing list