This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Ping: patch to remove the old RA
On Thu, Jan 29, 2009 at 10:35 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> 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?
Ok if this was bootstrapped & tested.
Thanks,
Richard.
> 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.
>