Re: [patch] Treegion formation for interblock scheduling

[Mike Stump <mrs at apple dot com>]

On Jun 14, 2006, at 1:46 PM, mcrosier@ncsu.edu wrote:
> +@item -fsched1-use-treegions
> +@opindex fsched1-use-treegions
> +When scheduling before register allocation, do use treegion  
> scheduling
> +algorithm.  Treegion scheduling allows motion across basic block  
> boundaries
> +resulting on faster schedules.

resulting in...

> +fsched1-use-treegions
> +Common Report Var(flag_sched1_use_treegions)
> +If scheduling pre reload, do treegion scheduling

Do treegion scheduling when -fschedule-insns or -O2 is given.

Would be better.

> +  while(!sbitmap_equal(saplings, zero_bitmap))

Whitespacing...

while (!sbitmap_equal (saplings, zero_bitmap))

> +    {
> +      /* Get a sapling to start a new treegion */
> +      node = sbitmap_first_set_bit(saplings);

Likewise.

> +      gcc_assert(node != ENTRY_BLOCK && node != EXIT_BLOCK);

Likewise.

> +      RESET_BIT(saplings, node);

Likewise.

> +      if(EDGE_COUNT(BASIC_BLOCK(node)->succs) > 0)

Likewise.

> +      while(head != tail)

Likewise.

> +	  while(!ei_end_p(current_ei))

Likewise.

> +	      if(EDGE_COUNT(ei_edge (current_ei)->dest->preds) > 1)

Likewise.

> +	      if(too_large_failure)

Likewise.

> +	      if(EDGE_COUNT(ei_edge (current_ei)->dest->succs) > 0)

Likewise.