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