Re: [PATCH] Change default of param not being smaller that min.

2017-02-22 Thread Jeff Law
On 02/21/2017 01:09 AM, Martin Liška wrote: On 02/21/2017 12:03 AM, Alexandre Oliva wrote: It's convenient to debug -fcompare-debug errors. You set the param to a large number, and then non-debug insns will get the same uid in both debug and non-debug compilations, so it's easier to compare RTL

Re: [PATCH] Change default of param not being smaller that min.

2017-02-21 Thread Martin Liška
On 02/21/2017 12:03 AM, Alexandre Oliva wrote: > It's convenient to debug -fcompare-debug errors. You set the param to a > large number, and then non-debug insns will get the same uid in both > debug and non-debug compilations, so it's easier to compare RTL dumps > (using the attached gcc-diff-dum

Re: [PATCH] Change default of param not being smaller that min.

2017-02-20 Thread Alexandre Oliva
On Feb 20, 2017, Richard Biener wrote: > On Fri, Feb 17, 2017 at 4:18 PM, Martin Liška wrote: >> Hello. >> >> We should not have a default value (different from -1) that is smaller than >> minimum. >> >> Ready to be installed after tests? > I think you should instead change the minimum value

Re: [PATCH] Change default of param not being smaller that min.

2017-02-20 Thread Richard Biener
On Fri, Feb 17, 2017 at 4:18 PM, Martin Liška wrote: > Hello. > > We should not have a default value (different from -1) that is smaller than > minimum. > > Ready to be installed after tests? I think you should instead change the minimum value. But OTOH I wonder why this is a --param at all...