RFC: LRA for x86/x86-64 [8/9]

Jeff Law law@redhat.com
Tue Oct 2 05:01:00 GMT 2012


On 09/27/2012 04:59 PM, Vladimir Makarov wrote:
>    The following patch adds a code neccessary for correct work of LRA
> (function ira_setup_eliminable_regset) and for correct work of the
> compiler when LRA is used (see file dwarf2out.c).
>
> 2012-09-27  Vladimir Makarov  <vmakarov@redhat.com>
>
>      * loop-invariant.c (calculate_loop_reg_pressure): Pass new
>      argument to ira_setup_eliminable_regset.
>      * haifa-sched.c (sched_init): Pass new argument to
>      ira_setup_eliminable_regset.
>      * dwarf2out.c: Include lra.h.
>      (based_loc_descr, compute_frame_pointer_to_fb_displacement): Use
>      lra_eliminate_regs for LRA instead of eliminate_regs.
>      * ira.c: (ira_setup_eliminable_regset): Add parameter. Remove
>      need_fp.  Call lra_init_elemination and mark
>      HARD_FRAME_POINTER_REGNUM as living forever if
>      frame_pointer_needed.
>      (ira): Call ira_setup_eliminable_regset with a new
>      argument.
>      * ira.h (ira_setup_eliminable_regset): Add an argument.
>      * Makefile.in (dwarf2out.o): Add dependence on ira.h and lra.h.
>
This is OK.  Obviously it's useless without 7a/7b.  But I just wanted to 
go ahead and review the ancillary bits before going to the meat of the 
submission.

So it's just the 7a/7b patch that needs review, right?

jeff



More information about the Gcc-patches mailing list