Re: [PATCH][3/4] Use RPO VN from unrolling

2018-08-01 Thread Richard Biener
On August 1, 2018 4:58:09 PM GMT+02:00, Richard Sandiford wrote: >Richard Biener writes: >> This should be 4/4 but I have the main patch on top, so... >> >> This uses the region-based VN from GIMPLE unrolling which means >> we better approximate the effects optimizations on unrolled inner >> loo

Re: [PATCH][3/4] Use RPO VN from unrolling

2018-08-01 Thread Richard Sandiford
Richard Biener writes: > This should be 4/4 but I have the main patch on top, so... > > This uses the region-based VN from GIMPLE unrolling which means > we better approximate the effects optimizations on unrolled inner > loops when evaluating whether to unroll outer ones. Great! Sounds like it

[PATCH][3/4] Use RPO VN from unrolling

2018-08-01 Thread Richard Biener
This should be 4/4 but I have the main patch on top, so... This uses the region-based VN from GIMPLE unrolling which means we better approximate the effects optimizations on unrolled inner loops when evaluating whether to unroll outer ones. * tree-ssa-loop-ivcanon.c: Include tree-ssa-sc