On Thu, Apr 21, 2011 at 8:43 AM, Richard Guenther
<richard.guent...@gmail.com> wrote:
> On Thu, Apr 21, 2011 at 5:22 PM, Michael Matz <m...@suse.de> wrote:
>> Hi,
>>
>> On Wed, 20 Apr 2011, Easwaran Raman wrote:
>>
>>> But you're right - not adding that conflict doesn't actually reduce the
>>> size of bit maps. Reverting back to what was there originally.
>>
>> Thanks, I have no more issues with the patch. You'll need to find someone
>> who can formally approve it, though.
>
> Ok with a proper changelog entry.
>
> Richard.
>
>>
>> Ciao,
>> Michael.
>>
>
Committed with the following Changelog entry:
2011-04-21 Easwaran Raman <era...@google.com>
* gcc/cfgexpand.c (stack_var): Remove OFFSET...
(add_stack_var): ...and its reference here...
(expand_stack_vars): ...and here.
(stack_var_cmp): Sort by descending order of size.
(partition_stack_vars): Change heuristic.
(union_stack_vars): Fix to reflect changes in
partition_stack_vars.
(dump_stack_var_partition): Add newline after each partition.
testsuite/Changelog:
2011-04-21 Easwaran Raman <era...@google.com>
* gcc.dg/stack-layout-2.c: New test.
Thanks,
Easwaran
