Re: Ping: patch to remove the old RA

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Thu, Jan 29, 2009 at 1:27 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Wed, Jan 28, 2009 at 2:12 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
>> The original patches were posted on
>>
>> http://gcc.gnu.org/ml/gcc-patches/2008-12/msg00613.html
>> http://gcc.gnu.org/ml/gcc-patches/2008-12/msg00741.html
>>
>> I am sending an updated version of the patch here because a few recent IRA
>> patches resulted in the patch updates too.
>>
>>  * Makefile.in (RA_H): Remove.
>
> Is ra.h still being used? If not, can we remove ra.h now?
>
> Also ira-color.c has
>
> /* Try to assign a hard register (except for FORBIDDEN_REGS) to
>   allocno A and return TRUE in the case of success.  That is an
>   analog of retry_global_alloc for IRA.  */
> static bool
> allocno_reload_assign (ira_allocno_t a, HARD_REG_SET forbidden_regs)
>
> Since retry_global_alloc is gone, this comment should be
> updated.
>

Like this patch.  OK for trunk?

Thanks.


-- 
H.J.
---
2009-01-29  H.J. Lu  <hongjiu.lu@intel.com>

	* ira-color.c (allocno_reload_assign): Update comments.
	* regmove.c (regmove_optimize): Likewise.

	* ra.h: Removed.

Attachment: gcc-foo-1.patch
Description: Text document