This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR19578: noreturn vs. function pointer propagation part 2
- From: Richard Sandiford <rsandifo at redhat dot com>
- To: Diego Novillo <dnovillo at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Wed, 02 Feb 2005 08:40:21 +0000
- Subject: Re: PR19578: noreturn vs. function pointer propagation part 2
- References: <87wtu5dbdb.fsf@redhat.com> <41F27889.1050102@redhat.com><87y8efkeq6.fsf_-_@redhat.com> <41F82EA6.60902@redhat.com><87oef637yj.fsf@firetop.home> <41FF8F11.7040007@redhat.com>
Hi Diego,
Thanks for the review.
Not objections, but genuine questions...
Diego Novillo <dnovillo@redhat.com> writes:
>> PR tree-optimization/19578
>> * tree-flow.h (modified_noreturn_calls): Declare.
>> (noreturn_call_p): Declare.
> >
> You need to initialize modified_noreturn_calls in init_tree_ssa and set
> it to NULL in delete_tree_ssa.
Why is it necessary to initialise modified_noreturn_calls? The VEC code
treats NULLs as empty vectors, so there doesn't seem to be any need to
initialise it explicitly. And why reserve an arbitrary number of slots
(like 20, or whatever) when many real-world functions won't use any?
I suppose setting it to null in delete_tree_ssa is just a way of making
sure that, if one pathological function has lots of noreturn calls,
the vector doesn't remain needlessly big for other functions.
Is that right?
Richard