[rfc] coalesce stack slots
Richard Henderson
rth@redhat.com
Fri Sep 3 07:52:00 GMT 2004
The following patch rearranges how local variables are expanded.
In particular, we do a (much) better job laying out stack variables
than in any previous gcc release.
As we're looking at all of the used locals, we set aside those that
we see must be allocated from the stack, as opposed to registers.
We construct an interference graph based on what we know of the
lifetimes of these variables. At the present time, this knowledge
is restricted to the scope in which the variable was declared. Which
can be non-optimial, but isn't bad except for the special case of
-ffloat-store.
We also include in the interference graph knowledge about how the
rtl-level alias analyzer works. Which does mean that with strict
aliasing on, variables with non-conflicting alias sets are given
distinct stack locations. There are enough testsuite cases that
fail without this to convince me this is unavoidable. Folks that
require absolute minimum stack usage will have to turn off strict
aliasing. Fortunately, kernel folk already use that option, because
kernels already do things that aren't by the book.
Finally, a standard partitioning scheme feeds a greedy binpacker
to actually lay out the objects in the stack frame. The good part
about this, from my perspective, is that the layout is completely
driven from the interference graph, so if someone decides at some
point to compute better lifetimes than the scope tree, it should
Just Work.
An open question is whether we should skip this work at -O0. The
computation time I have not measured, but I know that it *is*
quadratic in the number of variables allocated to the stack. Which
at -O0 is all of them excepting compiler temporaries. Alternately,
we could continue to coalesce large aggregates, and allocate scalars
and small aggregates immediately, not entering them into the
data structures.
There's also dead code to be removed from elsewhere in the compiler.
It's been tested on alphaev6 and i686 linux with no regressions.
I won't commit it just yet, pending commentary.
r~
PR middle-end/9997
* cfgexpand.c (LOCAL_ALIGNMENT): Provide default.
(STACK_ALIGNMENT_NEEDED, FRAME_GROWS_DOWNWARD): Likewise.
(struct stack_var, EOC, stack_vars, stack_vars_alloc, stack_vars_num,
stack_vars_sorted, stack_vars_conflict, stack_vars_conflict_alloc,
frame_phase, add_stack_var, triangular_index,
resize_stack_vars_conflict, add_stack_var_conflict,
stack_var_conflict_p, add_alias_set_conflicts, stack_var_size_cmp,
union_stack_vars, partition_stack_vars, dump_stack_var_partition,
expand_one_stack_var, expand_stack_vars, expand_one_static_var,
expand_one_hard_reg_var, expand_one_register_var,
expand_one_error_var, expand_one_var, expand_used_vars_for_block,
clear_tree_used): New.
(expand_used_vars): Rewrite.
Index: cfgexpand.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/cfgexpand.c,v
retrieving revision 2.18
diff -c -p -d -r2.18 cfgexpand.c
*** cfgexpand.c 23 Aug 2004 00:02:55 -0000 2.18
--- cfgexpand.c 3 Sep 2004 07:22:05 -0000
*************** Boston, MA 02111-1307, USA. */
*** 35,55 ****
#include "tree-pass.h"
#include "except.h"
#include "flags.h"
! /* Expand variables in the unexpanded_var_list. */
static void
expand_used_vars (void)
{
! tree cell;
! cfun->unexpanded_var_list = nreverse (cfun->unexpanded_var_list);
! for (cell = cfun->unexpanded_var_list; cell; cell = TREE_CHAIN (cell))
! expand_var (TREE_VALUE (cell));
cfun->unexpanded_var_list = NULL_TREE;
}
--- 35,733 ----
#include "tree-pass.h"
#include "except.h"
#include "flags.h"
+ #include "diagnostic.h"
+ #include "toplev.h"
! #ifndef LOCAL_ALIGNMENT
! #define LOCAL_ALIGNMENT(TYPE, ALIGNMENT) ALIGNMENT
! #endif
!
! #ifndef STACK_ALIGNMENT_NEEDED
! #define STACK_ALIGNMENT_NEEDED 1
! #endif
!
! #ifdef FRAME_GROWS_DOWNWARD
! # undef FRAME_GROWS_DOWNWARD
! # define FRAME_GROWS_DOWNWARD 1
! #else
! # define FRAME_GROWS_DOWNWARD 0
! #endif
!
!
! /* This structure holds data relevant to one variable that will be
! placed in a stack slot. */
! struct stack_var
! {
! /* The Variable. */
! tree decl;
!
! /* The offset of the variable. During partitioning, this is the
! offset relative to the partition. After partitioning, this
! is relative to the stack frame. */
! HOST_WIDE_INT offset;
!
! /* Initially, the size of the variable. Later, the size of the partition,
! if this variable becomes it's partition's representative. */
! HOST_WIDE_INT size;
!
! /* The *byte* alignment required for this variable. Or as, with the
! size, the alignment for this partition. */
! unsigned int alignb;
!
! /* The partition representative. */
! size_t representative;
!
! /* The next stack variable in the partition, or EOC. */
! size_t next;
! };
!
! #define EOC ((size_t)-1)
!
! /* We have an array of such objects while deciding allocation. */
! static struct stack_var *stack_vars;
! static size_t stack_vars_alloc;
! static size_t stack_vars_num;
!
! /* An array of indicies such that stack_vars[stack_vars_sorted[i]].size
! is non-decreasing. */
! static size_t *stack_vars_sorted;
!
! /* We have an interference graph between such objects. This graph
! is lower triangular. */
! static bool *stack_vars_conflict;
! static size_t stack_vars_conflict_alloc;
!
! /* The phase of the stack frame. This is the known misalignment of
! virtual_stack_vars_rtx from PREFERRED_STACK_BOUNDARY. That is,
! (frame_offset+frame_phase) % PREFERRED_STACK_BOUNDARY == 0. */
! static int frame_phase;
!
! /* Accumulate DECL into STACK_VARS. */
!
! static void
! add_stack_var (tree decl)
! {
! unsigned int align;
!
! if (stack_vars_num >= stack_vars_alloc)
! {
! if (stack_vars_alloc)
! stack_vars_alloc = stack_vars_alloc * 3 / 2;
! else
! stack_vars_alloc = 32;
! stack_vars
! = XRESIZEVEC (struct stack_var, stack_vars, stack_vars_alloc);
! }
! stack_vars[stack_vars_num].decl = decl;
! stack_vars[stack_vars_num].offset = 0;
! stack_vars[stack_vars_num].size = tree_low_cst (DECL_SIZE_UNIT (decl), 1);
!
! /* Discover the alignment to use for this decl. Ignore alignment we
! can't do with expected alignment of the boundary. */
! align = DECL_ALIGN (decl);
! align = LOCAL_ALIGNMENT (TREE_TYPE (decl), align);
! if (align > PREFERRED_STACK_BOUNDARY)
! align = PREFERRED_STACK_BOUNDARY;
! if (cfun->stack_alignment_needed < align)
! cfun->stack_alignment_needed = align;
! stack_vars[stack_vars_num].alignb = align / BITS_PER_UNIT;
!
! /* All variables are initially in their own partition. */
! stack_vars[stack_vars_num].representative = stack_vars_num;
! stack_vars[stack_vars_num].next = EOC;
!
! /* Ensure that this decl doesn't get put onto the list twice. */
! SET_DECL_RTL (decl, pc_rtx);
!
! stack_vars_num++;
! }
!
! /* Compute the linear index of a lower-triangular coordinate (I, J). */
!
! static size_t
! triangular_index (size_t i, size_t j)
! {
! if (i < j)
! {
! size_t t;
! t = i, i = j, j = t;
! }
! return (i * (i + 1)) / 2 + j;
! }
!
! /* Ensure that STACK_VARS_CONFLICT is large enough for N objects. */
!
! static void
! resize_stack_vars_conflict (size_t n)
! {
! size_t size = triangular_index (n-1, n-1) + 1;
!
! if (size <= stack_vars_conflict_alloc)
! return;
!
! stack_vars_conflict = XRESIZEVEC (bool, stack_vars_conflict, size);
! memset (stack_vars_conflict + stack_vars_conflict_alloc, 0,
! (size - stack_vars_conflict_alloc) * sizeof (bool));
! stack_vars_conflict_alloc = size;
! }
!
! /* Make the decls associated with luid's X and Y conflict. */
!
! static void
! add_stack_var_conflict (size_t x, size_t y)
! {
! size_t index = triangular_index (x, y);
! gcc_assert (index < stack_vars_conflict_alloc);
! stack_vars_conflict[index] = true;
! }
!
! /* Check whether the decls associated with luid's X and Y conflict. */
!
! static bool
! stack_var_conflict_p (size_t x, size_t y)
! {
! size_t index = triangular_index (x, y);
! gcc_assert (index < stack_vars_conflict_alloc);
! return stack_vars_conflict[index];
! }
!
! /* A subroutine of expand_used_vars. If two variables X and Y have alias
! sets that do not conflict, then do add a conflict for these variables
! in the interference graph. We also have to mind MEM_IN_STRUCT_P and
! MEM_SCALAR_P. */
!
! static void
! add_alias_set_conflicts (void)
! {
! size_t i, j, n = stack_vars_num;
!
! for (i = 0; i < n; ++i)
! {
! bool aggr_i = AGGREGATE_TYPE_P (TREE_TYPE (stack_vars[i].decl));
! HOST_WIDE_INT set_i = get_alias_set (stack_vars[i].decl);
!
! for (j = 0; j < i; ++j)
! {
! bool aggr_j = AGGREGATE_TYPE_P (TREE_TYPE (stack_vars[j].decl));
! HOST_WIDE_INT set_j = get_alias_set (stack_vars[j].decl);
! if (aggr_i != aggr_j || !alias_sets_conflict_p (set_i, set_j))
! add_stack_var_conflict (i, j);
! }
! }
! }
!
! /* A subroutine of partition_stack_vars. A comparison function for qsort,
! sorting an array of indicies by the size of the object. */
!
! static int
! stack_var_size_cmp (const void *a, const void *b)
! {
! HOST_WIDE_INT sa = stack_vars[*(const size_t *)a].size;
! HOST_WIDE_INT sb = stack_vars[*(const size_t *)b].size;
!
! if (sa < sb)
! return -1;
! if (sa > sb)
! return 1;
! return 0;
! }
!
! /* A subroutine of partition_stack_vars. The UNION portion of a UNION/FIND
! partitioning algorithm. Partitions A and B are known to be non-conflicting.
! Merge them into a single partition A.
!
! At the same time, add OFFSET to all variables in partition B. At the end
! of the partitioning process we've have a nice block easy to lay out within
! the stack frame. */
!
! static void
! union_stack_vars (size_t a, size_t b, HOST_WIDE_INT offset)
! {
! size_t i, last;
!
! /* Update each element of partition B with the given offset,
! and merge them into partition A. */
! for (last = i = b; i != EOC; last = i, i = stack_vars[i].next)
! {
! stack_vars[i].offset += offset;
! stack_vars[i].representative = a;
! }
! stack_vars[last].next = stack_vars[a].next;
! stack_vars[a].next = b;
!
! /* Update the required alignment of partition A to account for B. */
! if (stack_vars[a].alignb < stack_vars[b].alignb)
! stack_vars[a].alignb = stack_vars[b].alignb;
!
! /* Update the interference graph and merge the conflicts. */
! for (last = stack_vars_num, i = 0; i < last; ++i)
! if (stack_var_conflict_p (b, i))
! add_stack_var_conflict (a, i);
! }
!
! /* A subroutine of expand_used_vars. Binpack the variables into
! partitions constrained by the interference graph. The overall
! algorithm used is as follows:
!
! Sort the objects by size.
! For each object A {
! S = size(A)
! O = 0
! loop {
! Look for the largest non-conflicting object B with size <= S.
! UNION (A, B)
! offset(B) = O
! O += size(B)
! S -= size(B)
! }
! }
! */
!
! static void
! partition_stack_vars (void)
! {
! size_t si, sj, n = stack_vars_num;
!
! stack_vars_sorted = XNEWVEC (size_t, stack_vars_num);
! for (si = 0; si < n; ++si)
! stack_vars_sorted[si] = si;
!
! if (n == 1)
! return;
!
! qsort (stack_vars_sorted, n, sizeof (size_t), stack_var_size_cmp);
!
! /* Special case: detect when all variables conflict, and thus we can't
! do anything during the partitioning loop. It isn't uncommon (with
! C code at least) to declare all variables at the top of the function,
! and if we're not inlining, then all variables will be in the same scope.
! Take advantage of very fast libc routines for this scan. */
! gcc_assert (sizeof(bool) == sizeof(char));
! if (memchr (stack_vars_conflict, false, stack_vars_conflict_alloc) == NULL)
! return;
!
! for (si = 0; si < n; ++si)
! {
! size_t i = stack_vars_sorted[si];
! HOST_WIDE_INT isize = stack_vars[i].size;
! HOST_WIDE_INT offset = 0;
!
! for (sj = si; sj-- > 0; )
! {
! size_t j = stack_vars_sorted[sj];
! HOST_WIDE_INT jsize = stack_vars[j].size;
! unsigned int jalign = stack_vars[j].alignb;
!
! /* Ignore objects that aren't partition representatives. */
! if (stack_vars[j].representative != j)
! continue;
!
! /* Ignore objects too large for the remaining space. */
! if (isize < jsize)
! continue;
!
! /* Ignore conflicting objects. */
! if (stack_var_conflict_p (i, j))
! continue;
!
! /* Refine the remaining space check to include alignment. */
! if (offset & (jalign - 1))
! {
! HOST_WIDE_INT toff = offset;
! toff += jalign - 1;
! toff &= -(HOST_WIDE_INT)jalign;
! if (isize - (toff - offset) < jsize)
! continue;
!
! isize -= toff - offset;
! offset = toff;
! }
!
! /* UNION the objects, placing J at OFFSET. */
! union_stack_vars (i, j, offset);
!
! isize -= jsize;
! if (isize == 0)
! break;
! }
! }
! }
!
! /* A debugging aid for expand_used_vars. Dump the generated partitions. */
!
! static void
! dump_stack_var_partition (void)
! {
! size_t si, i, j, n = stack_vars_num;
!
! for (si = 0; si < n; ++si)
! {
! i = stack_vars_sorted[si];
!
! /* Skip variables that aren't partition representatives, for now. */
! if (stack_vars[i].representative != i)
! continue;
!
! fprintf (dump_file, "Partition %lu: size " HOST_WIDE_INT_PRINT_DEC
! " align %u\n", (unsigned long) i, stack_vars[i].size,
! stack_vars[i].alignb);
!
! for (j = i; j != EOC; j = stack_vars[j].next)
! {
! fputc ('\t', dump_file);
! print_generic_expr (dump_file, stack_vars[j].decl, dump_flags);
! fprintf (dump_file, ", offset " HOST_WIDE_INT_PRINT_DEC "\n",
! stack_vars[i].offset);
! }
! }
! }
!
! /* A subroutine of expand_stack_vars. Assign rtl to DECL at frame
! offset OFFSET. */
!
! static void
! expand_one_stack_var (tree decl, HOST_WIDE_INT offset)
! {
! HOST_WIDE_INT align;
! rtx x;
!
! /* If this fails, we've overflowed the stack frame. Error nicely? */
! gcc_assert (offset == trunc_int_for_mode (offset, Pmode));
!
! x = plus_constant (virtual_stack_vars_rtx, offset);
! x = gen_rtx_MEM (DECL_MODE (decl), x);
!
! /* Set alignment we actually gave this decl. */
! offset -= frame_phase;
! align = offset & -offset;
! align *= BITS_PER_UNIT;
! if (align > STACK_BOUNDARY || align == 0)
! align = STACK_BOUNDARY;
! DECL_ALIGN (decl) = align;
! DECL_USER_ALIGN (decl) = 0;
!
! set_mem_attributes (x, decl, true);
! SET_DECL_RTL (decl, x);
! }
!
! /* A subroutine of expand_used_vars. Give each partition representative
! a unique location within the stack frame. Update each partition member
! with that location. */
!
! static void
! expand_stack_vars (void)
! {
! size_t si, i, j, n = stack_vars_num;
! HOST_WIDE_INT cur_frame_offset = frame_offset;
!
! for (si = 0; si < n; ++si)
! {
! HOST_WIDE_INT offset, new_frame_offset;
! HOST_WIDE_INT size, align;
!
! i = stack_vars_sorted[si];
!
! /* Skip variables that aren't partition representatives, for now. */
! if (stack_vars[i].representative != i)
! continue;
!
! size = stack_vars[i].size;
! align = stack_vars[i].alignb;
!
! /* Allocate space within the stack frame for the entire partition. */
! new_frame_offset = cur_frame_offset;
! if (FRAME_GROWS_DOWNWARD)
! {
! new_frame_offset -= size + frame_phase;
! new_frame_offset &= -align;
! new_frame_offset += frame_phase;
! offset = new_frame_offset;
! }
! else
! {
! new_frame_offset -= frame_phase;
! new_frame_offset += align - 1;
! new_frame_offset &= -align;
! new_frame_offset += frame_phase;
! offset = new_frame_offset;
! new_frame_offset += size;
! }
! cur_frame_offset = new_frame_offset;
!
! /* Create rtl for each variable based on their location within the
! partition. */
! for (j = i; j != EOC; j = stack_vars[j].next)
! expand_one_stack_var (stack_vars[j].decl,
! stack_vars[j].offset + offset);
! }
!
! /* If the target requires that FRAME_OFFSET be aligned, do it. */
! if (STACK_ALIGNMENT_NEEDED)
! {
! HOST_WIDE_INT align = PREFERRED_STACK_BOUNDARY / BITS_PER_UNIT;
! if (!FRAME_GROWS_DOWNWARD)
! cur_frame_offset += align - 1;
! cur_frame_offset = cur_frame_offset & -align;
! }
! frame_offset = cur_frame_offset;
! }
!
! /* A subroutine of expand_one_var. Called to assign rtl
! to a TREE_STATIC VAR_DECL. */
!
! static void
! expand_one_static_var (tree var)
! {
! /* If this is an inlined copy of a static local variable,
! look up the original. */
! var = DECL_ORIGIN (var);
!
! /* If we've already processed this variable because of that, do nothing. */
! if (TREE_ASM_WRITTEN (var))
! return;
!
! /* Give the front end a chance to do whatever. In practice, this is
! resolving duplicate names for IMA in C. */
! if (lang_hooks.expand_decl (var))
! return;
!
! /* Otherwise, just emit the variable. */
! rest_of_decl_compilation (var, 0, 0);
! }
!
! /* A subroutine of expand_one_var. Called to assign rtl to a VAR_DECL
! that will reside in a hard register. */
!
! static void
! expand_one_hard_reg_var (tree var)
! {
! rest_of_decl_compilation (var, 0, 0);
! }
!
! /* A subroutine of expand_one_var. Called to assign rtl to a VAR_DECL
! that will reside in a pseudo register. */
!
! static void
! expand_one_register_var (tree var)
! {
! tree type = TREE_TYPE (var);
! int unsignedp = TYPE_UNSIGNED (type);
! enum machine_mode reg_mode
! = promote_mode (type, DECL_MODE (var), &unsignedp, 0);
! rtx x = gen_reg_rtx (reg_mode);
!
! SET_DECL_RTL (var, x);
!
! /* Note if the object is a user variable. */
! if (!DECL_ARTIFICIAL (var))
! {
! mark_user_reg (x);
!
! /* Trust user variables which have a pointer type to really
! be pointers. Do not trust compiler generated temporaries
! as our type system is totally busted as it relates to
! pointer arithmetic which translates into lots of compiler
! generated objects with pointer types, but which are not really
! pointers. */
! if (POINTER_TYPE_P (type))
! mark_reg_pointer (x, TYPE_ALIGN (TREE_TYPE (TREE_TYPE (var))));
! }
! }
!
! /* A subroutine of expand_one_var. Called to assign rtl to a VAR_DECL that
! has some associated error, e.g. it's type is error-mark. We just need
! to pick something that won't crash the rest of the compiler. */
!
! static void
! expand_one_error_var (tree var)
! {
! enum machine_mode mode = DECL_MODE (var);
! rtx x;
!
! if (mode == BLKmode)
! x = gen_rtx_MEM (BLKmode, const0_rtx);
! else if (mode == VOIDmode)
! x = const0_rtx;
! else
! x = gen_reg_rtx (mode);
!
! SET_DECL_RTL (var, x);
! }
!
! /* A subroutine of expand_used_vars. Expand one variable according to
! its flavour. Variables to be placed on the stack are not actually
! expanded yet, merely recorded. */
!
! static void
! expand_one_var (tree var)
! {
! if (TREE_CODE (var) != VAR_DECL)
! lang_hooks.expand_decl (var);
! else if (DECL_EXTERNAL (var))
! ;
! else if (DECL_VALUE_EXPR (var))
! ;
! else if (TREE_STATIC (var))
! expand_one_static_var (var);
! else if (DECL_RTL_SET_P (var))
! ;
! else if (TREE_TYPE (var) == error_mark_node)
! expand_one_error_var (var);
! else if (DECL_HARD_REGISTER (var))
! expand_one_hard_reg_var (var);
! else if (use_register_for_decl (var))
! expand_one_register_var (var);
! else
! add_stack_var (var);
! }
!
! /* A subroutine of expand_used_vars. Walk down through the BLOCK tree
! expanding variables. Those variables that can be put into registers
! are allocated pseudos; those that can't are put on the stack.o
!
! OLD_SV_NUM is the value of STACK_VARS_NUM on entry to the function,
! or 0 in the special case of the outermost BLOCK. */
!
! static void
! expand_used_vars_for_block (tree block, size_t old_sv_num)
! {
! size_t i, j, this_sv_num, new_sv_num;
! tree t;
!
! /* Expand all variables at this level. */
! for (t = BLOCK_VARS (block); t ; t = TREE_CHAIN (t))
! if (TREE_USED (t))
! expand_one_var (t);
!
! this_sv_num = stack_vars_num;
!
! /* Expand all variables at containing levels. */
! for (t = BLOCK_SUBBLOCKS (block); t ; t = BLOCK_CHAIN (t))
! expand_used_vars_for_block (t, stack_vars_num);
!
! /* Since we do not track exact variable lifetimes (which is not even
! possible for varibles whose address escapes), we mirror the block
! tree in the interference graph. Here we cause all variables at this
! level, and all sublevels, to conflict. Do make certain that a
! variable conflicts with itself. */
! if (old_sv_num < this_sv_num)
! {
! new_sv_num = stack_vars_num;
! resize_stack_vars_conflict (new_sv_num);
!
! for (i = old_sv_num; i < new_sv_num; ++i)
! for (j = i < this_sv_num ? i : this_sv_num; ; --j)
! {
! add_stack_var_conflict (i, j);
! if (j == old_sv_num)
! break;
! }
! }
! }
!
! /* A subroutine of expand_used_vars. Walk down through the BLOCK tree
! and clear TREE_USED on all local variables. */
!
! static void
! clear_tree_used (tree block)
! {
! tree t;
!
! for (t = BLOCK_VARS (block); t ; t = TREE_CHAIN (t))
! /* if (!TREE_STATIC (t) && !DECL_EXTERNAL (t)) */
! TREE_USED (t) = 0;
!
! for (t = BLOCK_SUBBLOCKS (block); t ; t = BLOCK_CHAIN (t))
! clear_tree_used (t);
! }
!
! /* Expand all variables used in the function. */
static void
expand_used_vars (void)
{
! tree t, outer_block = DECL_INITIAL (current_function_decl);
! /* Compute the phase of the stack frame for this function. */
! {
! int align = PREFERRED_STACK_BOUNDARY / BITS_PER_UNIT;
! int off = STARTING_FRAME_OFFSET % align;
! frame_phase = off ? align - off : 0;
! }
! /* Set TREE_USED on all variables in the unexpanded_var_list. */
! for (t = cfun->unexpanded_var_list; t; t = TREE_CHAIN (t))
! TREE_USED (TREE_VALUE (t)) = 1;
!
! /* Clear TREE_USED on all variables associated with a block scope. */
! clear_tree_used (outer_block);
!
! /* At this point all variables on the unexpanded_var_list with TREE_USED
! set are not associated with any block scope. Lay them out. */
! for (t = cfun->unexpanded_var_list; t; t = TREE_CHAIN (t))
! {
! tree var = TREE_VALUE (t);
! bool expand_now = false;
!
! /* We didn't set a block for static or extern because it's hard
! to tell the difference between a global variable (re)declared
! in a local scope, and one that's really declared there to
! begin with. And it doesn't really matter much, since we're
! not giving them stack space. Expand them now. */
! if (TREE_STATIC (var) || DECL_EXTERNAL (var))
! expand_now = true;
!
! /* Any variable that could have been hoisted into an SSA_NAME
! will have been propagated anywhere the optimizers chose,
! i.e. not confined to their original block. Allocate them
! as if they were defined in the outermost scope. */
! else if (is_gimple_reg (var))
! expand_now = true;
!
! /* If the variable is not associated with any block, then it
! was created by the optimizers, and could be live anywhere
! in the function. */
! else if (TREE_USED (var))
! expand_now = true;
!
! /* Finally, mark all variables on the list as used. We'll use
! this in a moment when we expand those associated with scopes. */
! TREE_USED (var) = 1;
+ if (expand_now)
+ expand_one_var (var);
+ }
cfun->unexpanded_var_list = NULL_TREE;
+
+ /* At this point, all variables within the block tree with TREE_USED
+ set are actually used by the optimized function. Lay them out. */
+ expand_used_vars_for_block (outer_block, 0);
+ if (stack_vars_num == 0)
+ return;
+
+ /* Due to the way alias sets work, no variables with non-conflicting
+ alias sets may be assigned the same address. Add conflicts to
+ reflect this. */
+ add_alias_set_conflicts ();
+
+ /* Now that we have collected all stack variables, and have computed a
+ minimal interference graph, attempt to save some stack space. */
+ partition_stack_vars ();
+ if (dump_file)
+ dump_stack_var_partition ();
+
+ /* Assign rtl to each variable based on these partitions. */
+ expand_stack_vars ();
+
+ /* Free up stack variable graph data. */
+ XDELETEVEC (stack_vars);
+ XDELETEVEC (stack_vars_sorted);
+ XDELETEVEC (stack_vars_conflict);
+ stack_vars = NULL;
+ stack_vars_alloc = stack_vars_num = 0;
+ stack_vars_conflict = NULL;
+ stack_vars_conflict_alloc = 0;
}
More information about the Gcc-patches
mailing list