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]

Re: [patches] Re: Allow updating of loglinks.


> On Wed, Oct 17, 2001 at 04:28:39PM +0200, Jan Hubicka wrote:
> > + 	      LOG_LINKS (insn) = 0;
> 
> Should use free_INSN_LIST_list.  Perhaps you want to use
> clear_log_links instead?  Or modify clear_log_links to
> honor the blocks argument and split out the global life
> clearing to a different function.

What I was concerned about is that if we clear log links before
building, we can drop the clear_log_links.
This looks better to me, as it requires fewer care.

I didn't it in the same patch, because clear_log_links does also clearing of
life infromation, so perhas it should be called "clear_life_info" instead.
Does that make sense?

BTW the clear_log_links also don't use free_INSN_LIST_list.

Honza
> 
> 
> r~


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