On Wed, Sep 5, 2018 at 10:31 AM Martin Liška <mli...@suse.cz> wrote: > > Hi. > > It's beneficial to group cases before switch lowering machinery > is making a decision what to do with a switch. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed?
Hmm, do we want to do this at O0? There it should be redundant given CFG build already performs it? We also perform it at .optimized (execute_cleanup_cfg_post_optimizing) plus when we redirect some edges and then call record_case_labels. Otherwise OK. Richard. > Martin > > > gcc/ChangeLog: > > 2018-09-04 Martin Liska <mli...@suse.cz> > > PR tree-optimization/87205 > * tree-switch-conversion.c (pass_lower_switch::execute): > Group cases for switch statements. > > gcc/testsuite/ChangeLog: > > 2018-09-04 Martin Liska <mli...@suse.cz> > > PR tree-optimization/87205 > * gcc.dg/tree-ssa/pr87205-2.c: New test. > * gcc.dg/tree-ssa/pr87205.c: New test. > --- > gcc/testsuite/gcc.dg/tree-ssa/pr87205-2.c | 18 ++++++++++++++++++ > gcc/testsuite/gcc.dg/tree-ssa/pr87205.c | 21 +++++++++++++++++++++ > gcc/tree-switch-conversion.c | 8 ++++++-- > 3 files changed, 45 insertions(+), 2 deletions(-) > create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr87205-2.c > create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr87205.c > >
