Re: [PATCH] Use RPO order for fwprop iteration

2016-09-02 Thread Richard Biener
On Fri, 2 Sep 2016, Robin Dapp wrote: > This causes a performance regression in the xalancbmk SPECint2006 > benchmark on s390x. At first sight, the produced asm output doesn't look > too different but I'll have a closer look. Is the fwprop order supposed > to have major performance implications?

Re: [PATCH] Use RPO order for fwprop iteration

2016-09-02 Thread Robin Dapp
This causes a performance regression in the xalancbmk SPECint2006 benchmark on s390x. At first sight, the produced asm output doesn't look too different but I'll have a closer look. Is the fwprop order supposed to have major performance implications? Regards Robin > This changes it from PRE on t

[PATCH] Use RPO order for fwprop iteration

2016-08-22 Thread Richard Biener
This changes it from PRE on the inverted graph to RPO order which works better for loops and blocks with no path to exit. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2016-08-22 Richard Biener * tree-ssa-forwprop.c (pass_forwprop::execute): Use RPO order.