This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA][lto merge]: support for cgraph node sets

On Fri, Apr 17, 2009 at 17:14, Tom Tromey <tromey@redhat.com> wrote:
>>>>>> "Diego" == Diego Novillo <dnovillo@google.com> writes:
>
> Diego> +#ifdef ENABLE_CHECKING
> Diego> + Â Â Âelement = (cgraph_node_set_element) *slot;
> Diego> + Â Â Âgcc_assert (node == element->node
> Diego> + Â Â Â Â Â Â Â Â Â&& (VEC_index (cgraph_node_ptr, set->nodes, element->index)
> Diego> + Â Â Â Â Â Â Â Â Â Â Â== node));
> Diego> +#endif
>
> Is there a particular reason to wrap these assertions in
> ENABLE_CHECKING? ÂThe expressions seem cheap enough that you might as
> well make them unconditional.

Good point.  I'll change it.


Thanks.  Diego.
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]