Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Richard Guenther
On Mon, Jul 16, 2012 at 11:03 AM, Steven Bosscher wrote: > On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther > wrote: >> Maybe we should rename EDGE_CAN_FALLTHRU to EDGE_PLF_1 >> and have a #define in bb-reorder.c then? Other passes might use >> it for their pass-local handling then and would n

Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Steven Bosscher
On Mon, Jul 16, 2012 at 10:53 AM, Richard Guenther wrote: > Maybe we should rename EDGE_CAN_FALLTHRU to EDGE_PLF_1 > and have a #define in bb-reorder.c then? Other passes might use > it for their pass-local handling then and would not be content to set/unset > it. I intend to do a lot more clean

Re: [patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-16 Thread Richard Guenther
On Sat, Jul 14, 2012 at 11:40 PM, Steven Bosscher wrote: > Hello, > > EDGE_CAN_FALLTHRU is internal to bb-reorder.c, > set_edge_can_fallthru_flag() clears it on all edges and computes it > fresh for bb-reorder. > > The test in rtl_verify_flow_info_1 has to stay because bb-reorder > doesn't clean u

[patch] Remove useless sets/clears of EDGE_CAN_FALLTHRU

2012-07-14 Thread Steven Bosscher
Hello, EDGE_CAN_FALLTHRU is internal to bb-reorder.c, set_edge_can_fallthru_flag() clears it on all edges and computes it fresh for bb-reorder. The test in rtl_verify_flow_info_1 has to stay because bb-reorder doesn't clean up the flag after it's done. Bootstrapped&tested on powerpc64-unknown-li