Re: [patch] PR53881 - again

2012-07-23 Thread Richard Guenther
On Mon, Jul 23, 2012 at 1:23 AM, Steven Bosscher wrote: > Hello, > > This patch fixes PR53881 by making group_case_labels_stmt look at the > CFG instead of relying on label equality. > > Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK? Ok. Thanks, Richard. > Ciao! > Steven

[patch] PR53881 - again

2012-07-22 Thread Steven Bosscher
Hello, This patch fixes PR53881 by making group_case_labels_stmt look at the CFG instead of relying on label equality. Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK? Ciao! Steven gcc/ PR tree-optimization/53881 * tree-cfg.c (group_case_labels_stmt): Look up the basic blo

[patch] PR53881

2012-07-07 Thread Steven Bosscher
Hello, I was relying on the assumption that all case labels in a switch vector that jump to the same target block would also have the same CASE_LABEL. This isn't so for the test case of PR53881, where DCE removes a dead statement but preserves a user label. Fortunately, using find_edge in emit_cas