On 08/09/15 23:08, Jeff Law wrote:
On 09/08/2015 07:21 AM, Tom de Vries wrote:
[ was: Re: [RFC] Prevent unnecessary recompilation for trivial
params.def changes ]

On 08/09/15 14:03, Andreas Schwab wrote:
Tom de Vries <tom_devr...@mentor.com> writes:

After a subsequent rebuild I don't see anything being rebuild. So I
don't
observe 'continuous rebuilding'.

What happens when you just touch params-list.h or params.def?
move-if-change will leave the target untouched when unchanged (that's
the whole point of it), so it will remain older than the dependencies.

I could reproduce the problem using these instructions, thanks.

I also found a bit "On the use of stamps" in gcc/Makefile.in, which
explains the problem and how to fix things.

Updated patch accordingly.

OK for trunk if bootstrap succeeds?
Yes.

This patch adds the missing copyright notice in params-list.h.

Committed as trivial.

Thanks,
- Tom
Add copyright in gcc/params-list.h

2015-09-09  Tom de Vries  <t...@codesourcery.com>

	* params-list.h: Add missing copyright notice.
---
 gcc/params-list.h | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gcc/params-list.h b/gcc/params-list.h
index 49301d2..ee33ef5 100644
--- a/gcc/params-list.h
+++ b/gcc/params-list.h
@@ -1,3 +1,22 @@
+/* File used to generate params.list
+   Copyright (C) 2015 Free Software Foundation, Inc.
+
+This file is part of GCC.
+
+GCC is free software; you can redistribute it and/or modify it under
+the terms of the GNU General Public License as published by the Free
+Software Foundation; either version 3, or (at your option) any later
+version.
+
+GCC is distributed in the hope that it will be useful, but WITHOUT ANY
+WARRANTY; without even the implied warranty of MERCHANTABILITY or
+FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+for more details.
+
+You should have received a copy of the GNU General Public License
+along with GCC; see the file COPYING3.  If not see
+<http://www.gnu.org/licenses/>.  */
+
 #define DEFPARAM(enumerator, option, nocmsgid, default, min, max) \
   enumerator,
 #include "params.def"
-- 
1.9.1

Reply via email to