Message ID | 2e731a26-46b8-2948-352e-c5db2815735c@suse.cz |
---|---|
State | New |
Headers | show |
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 dumps >> (using the attached gcc-diff-dump; I thought it was in contrib, but it's >> not; should I put it there?) and locate the differences. > Yes, please commit the script, I'm interested in :) > > There's second version, which changes default to 0. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > Martin > > > 0001-Change-default-of-param-not-being-smaller-that-min.patch > > > From c4d196cb9ad7ace112e9013403323fea3ead69fc Mon Sep 17 00:00:00 2001 > From: marxin <mliska@suse.cz> > Date: Fri, 17 Feb 2017 16:00:30 +0100 > Subject: [PATCH] Change default of param not being smaller that min. > > gcc/ChangeLog: > > 2017-02-17 Martin Liska <mliska@suse.cz> > > * params.def (PARAM_MIN_NONDEBUG_INSN_UID): Change default to 0. OK. jeff
From c4d196cb9ad7ace112e9013403323fea3ead69fc Mon Sep 17 00:00:00 2001 From: marxin <mliska@suse.cz> Date: Fri, 17 Feb 2017 16:00:30 +0100 Subject: [PATCH] Change default of param not being smaller that min. gcc/ChangeLog: 2017-02-17 Martin Liska <mliska@suse.cz> * params.def (PARAM_MIN_NONDEBUG_INSN_UID): Change default to 0. --- gcc/params.def | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/params.def b/gcc/params.def index 023ca727648..31847efbc3b 100644 --- a/gcc/params.def +++ b/gcc/params.def @@ -965,7 +965,7 @@ DEFPARAM (PARAM_MAX_VARTRACK_REVERSE_OP_SIZE, DEFPARAM (PARAM_MIN_NONDEBUG_INSN_UID, "min-nondebug-insn-uid", "The minimum UID to be used for a nondebug insn.", - 0, 1, 0) + 0, 0, 0) DEFPARAM (PARAM_IPA_SRA_PTR_GROWTH_FACTOR, "ipa-sra-ptr-growth-factor", -- 2.11.0