Re: [google] Backport r172837 and r172788 to google/main

2011-05-06 Thread Diego Novillo
On Fri, May 6, 2011 at 14:22, Easwaran Raman era...@google.com wrote:
 Backported r172788 and r172837  from trunk to google/main.


Minor nit:

 2011-05-06  Easwaran Raman  era...@google.com

 Backport r172837:


This needs to be indented by 1 tab.

        * cfgexpand.c (stack_var): Remove OFFSET...
        (add_stack_var): ...and its reference here...
        (expand_stack_vars): ...and here.


Diego.


Re: [google] Backport r172837 and r172788 to google/main

2011-05-06 Thread Easwaran Raman
Thanks. Fixed them (as well as the same issue in some earlier entries).

-Easwaran

On Fri, May 6, 2011 at 11:27 AM, Diego Novillo dnovi...@google.com wrote:
 On Fri, May 6, 2011 at 14:22, Easwaran Raman era...@google.com wrote:
 Backported r172788 and r172837  from trunk to google/main.


 Minor nit:

 2011-05-06  Easwaran Raman  era...@google.com

 Backport r172837:


 This needs to be indented by 1 tab.

        * cfgexpand.c (stack_var): Remove OFFSET...
        (add_stack_var): ...and its reference here...
        (expand_stack_vars): ...and here.


 Diego.



Re: [google] Backport r172837 and r172788 to google/main

2011-05-06 Thread Diego Novillo
On Fri, May 6, 2011 at 14:34, Easwaran Raman era...@google.com wrote:
 Thanks. Fixed them (as well as the same issue in some earlier entries).

Great, thanks!