Re: [PATCH] More PR92046 fixes, make --param allow-store-data-races a -f option

2019-10-15 Thread Jakub Jelinek
On Tue, Oct 15, 2019 at 01:21:12PM +0200, Richard Biener wrote: > On Tue, 15 Oct 2019, Kyrill Tkachov wrote: > > > Hi Richard, > > > > On 10/15/19 8:17 AM, Richard Biener wrote: > > > > > > This makes allow-store-data-races adjustable per function by making it > > > a regular option rather than a

Re: [PATCH] More PR92046 fixes, make --param allow-store-data-races a -f option

2019-10-15 Thread Richard Biener
On Tue, 15 Oct 2019, Kyrill Tkachov wrote: > Hi Richard, > > On 10/15/19 8:17 AM, Richard Biener wrote: > > > > This makes allow-store-data-races adjustable per function by making it > > a regular option rather than a --param. > > > Note that the kernel has --param=allow-store-data-races=0 in i

Re: [PATCH] More PR92046 fixes, make --param allow-store-data-races a -f option

2019-10-15 Thread Kyrill Tkachov
Hi Richard, On 10/15/19 8:17 AM, Richard Biener wrote: This makes allow-store-data-races adjustable per function by making it a regular option rather than a --param. Note that the kernel has --param=allow-store-data-races=0 in its build flags. I guess that will break unless they rename it

Re: [PATCH] More PR92046 fixes, make --param allow-store-data-races a -f option

2019-10-15 Thread Jakub Jelinek
On Tue, Oct 15, 2019 at 09:17:00AM +0200, Richard Biener wrote: > > This makes allow-store-data-races adjustable per function by making it > a regular option rather than a --param. > > Bootstrapped and tested on x86_64-unknown-linux-gnu, OK? LGTM. > 2019-10-15 Richard Biener > > PR mi

[PATCH] More PR92046 fixes, make --param allow-store-data-races a -f option

2019-10-15 Thread Richard Biener
This makes allow-store-data-races adjustable per function by making it a regular option rather than a --param. Bootstrapped and tested on x86_64-unknown-linux-gnu, OK? Thanks, Richard. 2019-10-15 Richard Biener PR middle-end/92046 * common.opt (fallow-store-data-races): New

[PATCH] More PR92046 fixes

2019-10-14 Thread Richard Biener
Two other params do similar scaling so fix them similarly. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2019-10-14 Richard Biener PR middle-end/92046 * dse.c (scan_insn): Use param max_active_local_stores. (dse_step1): Get PARAM_MAX_DSE_AC