Ping: unreviewed copyright patches
Not exactly the most exciting patches, and certainly not worth more than one ping, but: fixincludes copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00442.html http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00443.html libgcc copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00642.html libdecnumber copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00647.html Thanks, Richard
RFA: Update copyright for libitm
Update the copyright notices for libitm. Tested on x86_64-linux-gnu. OK to install? Thanks, Richard libitm/ Update copyright years Index: libitm/aatree.cc === --- libitm/aatree.cc2013-01-24 19:02:50.006757650 + +++ libitm/aatree.cc2013-02-02 10:01:22.911148693 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/aatree.h === --- libitm/aatree.h 2013-01-24 19:02:50.006757650 + +++ libitm/aatree.h 2013-02-02 10:01:22.912148698 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/alloc.cc === --- libitm/alloc.cc 2013-01-24 19:02:50.006757650 + +++ libitm/alloc.cc 2013-02-02 10:01:22.918148736 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/alloc_c.cc === --- libitm/alloc_c.cc 2013-01-24 19:02:50.006757650 + +++ libitm/alloc_c.cc 2013-02-02 10:01:22.918148736 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/alloc_cpp.cc === --- libitm/alloc_cpp.cc 2013-01-24 19:02:50.006757650 + +++ libitm/alloc_cpp.cc 2013-02-02 10:01:22.918148736 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/barrier.cc === --- libitm/barrier.cc 2013-01-24 19:02:50.006757650 + +++ libitm/barrier.cc 2013-02-02 10:01:22.919148741 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2008-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/beginend.cc === --- libitm/beginend.cc 2013-01-24 19:02:50.005757643 + +++ libitm/beginend.cc 2013-02-02 10:01:22.919148741 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009, 2011, 2012 Free Software Foundation, Inc. +/* Copyright (C) 2008-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/clone.cc === --- libitm/clone.cc 2013-01-24 19:02:50.005757643 + +++ libitm/clone.cc 2013-02-02 10:01:22.920148748 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2010, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/common.h === --- libitm/common.h 2013-01-24 19:02:50.006757650 + +++ libitm/common.h 2013-02-02 10:01:22.920148748 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2008-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index: libitm/config/alpha/cacheline.h === --- libitm/config/alpha/cacheline.h 2013-01-24 19:02:50.004757636 + +++ libitm/config/alpha/cacheline.h 2013-02-02 10:01:22.945148899 + @@ -1,4 +1,4 @@ -/* Copyright (C) 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2009-2013 Free Software Foundation, Inc. Contributed by Richard Henderson r...@redhat.com. This file is part of the GNU Transactional Memory Library (libitm). Index:
RFA: Update copyright for libmudflap
Update the copyright for libmudflap, in the same way as for other directories. Tested on x86_64-linux-gnu. OK to install? Richard libmudflap/ Update copyright years. Index: libmudflap/Makefile.am === --- libmudflap/Makefile.am 2013-02-02 21:33:00.753873985 + +++ libmudflap/Makefile.am 2013-02-03 10:27:57.961050269 + @@ -1,7 +1,6 @@ ## Makefile for the toplevel directory of the mudflap library. ## -## Copyright (C) 2002, 2003, 2004, 2007 -## Free Software Foundation, Inc. +## Copyright (C) 2002-2013 Free Software Foundation, Inc. ## AUTOMAKE_OPTIONS = 1.8 foreign Index: libmudflap/mf-heuristics.c === --- libmudflap/mf-heuristics.c 2013-02-02 21:33:00.754873993 + +++ libmudflap/mf-heuristics.c 2013-02-03 10:27:58.140051076 + @@ -1,5 +1,5 @@ /* Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2009 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/mf-hooks1.c === --- libmudflap/mf-hooks1.c 2013-02-02 21:33:00.754873993 + +++ libmudflap/mf-hooks1.c 2013-02-03 10:27:58.016050517 + @@ -1,5 +1,5 @@ /* Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2009, 2011 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/mf-hooks2.c === --- libmudflap/mf-hooks2.c 2013-02-02 21:33:00.754873993 + +++ libmudflap/mf-hooks2.c 2013-02-03 10:27:58.203051359 + @@ -1,5 +1,5 @@ /* Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2009, 2011 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/mf-hooks3.c === --- libmudflap/mf-hooks3.c 2013-02-02 21:33:00.754873993 + +++ libmudflap/mf-hooks3.c 2013-02-03 10:27:58.051050675 + @@ -1,6 +1,5 @@ /* Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2005, 2009, 2011 - Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/mf-impl.h === --- libmudflap/mf-impl.h2013-02-02 21:33:00.753873985 + +++ libmudflap/mf-impl.h2013-02-03 10:27:57.890049949 + @@ -1,6 +1,6 @@ /* Implementation header for mudflap runtime library. Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2009, 2011 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/mf-runtime.c === --- libmudflap/mf-runtime.c 2013-02-02 21:33:00.754873993 + +++ libmudflap/mf-runtime.c 2013-02-03 10:27:58.105050919 + @@ -1,6 +1,5 @@ /* Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2005, 2006, 2008, 2009, 2010, 2011, 2012, - 2013 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Splay Tree code originally by Mark Mitchell m...@markmitchell.com, Index: libmudflap/mf-runtime.h === --- libmudflap/mf-runtime.h 2013-02-02 21:33:00.753873985 + +++ libmudflap/mf-runtime.h 2013-02-03 10:27:57.927050117 + @@ -1,6 +1,6 @@ /* Implementation header for mudflap runtime library. Mudflap: narrow-pointer bounds-checking by tree rewriting. - Copyright (C) 2002, 2003, 2004, 2005, 2009 Free Software Foundation, Inc. + Copyright (C) 2002-2013 Free Software Foundation, Inc. Contributed by Frank Ch. Eigler f...@redhat.com and Graydon Hoare gray...@redhat.com Index: libmudflap/testsuite/lib/libmudflap.exp === --- libmudflap/testsuite/lib/libmudflap.exp 2013-02-02 21:33:00.753873985 + +++ libmudflap/testsuite/lib/libmudflap.exp 2013-02-03 10:27:58.286051733 + @@ -1,5
RFA: Update copyright for libobjc
Update the copyright notices for libobjc. Tested on x86_64-linux-gnu. OK to install? Richard libbjc/ Update copyright years. Index: libobjc/Makefile.in === --- libobjc/Makefile.in 2013-02-02 21:33:00.727873787 + +++ libobjc/Makefile.in 2013-02-03 10:31:00.272954078 + @@ -1,6 +1,5 @@ # Makefile for GNU Objective C runtime library. -# Copyright 1993, 1995, 1996, 1997, 1998, 1999, 2001, 2002, 2003, 2004, -# 2005, 2006, 2007, 2008, 2009, 2010, 2011 Free Software Foundation, Inc. +# Copyright (C) 1993-2013 Free Software Foundation, Inc. #This file is part of GCC. Index: libobjc/NXConstStr.m === --- libobjc/NXConstStr.m2013-02-02 21:33:00.724873765 + +++ libobjc/NXConstStr.m2013-02-03 10:31:00.225953831 + @@ -1,5 +1,5 @@ /* Implementation of the NXConstantString class for Objective-C. - Copyright (C) 1995, 2009, 2010 Free Software Foundation, Inc. + Copyright (C) 1995-2013 Free Software Foundation, Inc. Contributed by Pieter J. Schoenmakers ti...@es.ele.tue.nl This file is part of GCC. Index: libobjc/Object.m === --- libobjc/Object.m2013-02-02 21:33:00.728873795 + +++ libobjc/Object.m2013-02-03 10:31:00.226953836 + @@ -1,6 +1,5 @@ /* The implementation of class Object for Objective-C. - Copyright (C) 1993, 1994, 1995, 1997, 2002, 2009, 2010, 2011 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. This file is part of GCC. Index: libobjc/Protocol.m === --- libobjc/Protocol.m 2013-02-02 21:33:00.724873765 + +++ libobjc/Protocol.m 2013-02-03 10:31:00.243953925 + @@ -1,5 +1,5 @@ /* This file contains the implementation of class Protocol. - Copyright (C) 1993, 2004, 2009, 2010, 2011 Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. This file is part of GCC. Index: libobjc/accessors.m === --- libobjc/accessors.m 2013-02-02 21:33:00.725873772 + +++ libobjc/accessors.m 2013-02-03 10:31:00.272954078 + @@ -1,5 +1,5 @@ /* GNU Objective C Runtime accessors functions - Copyright (C) 2010 Free Software Foundation, Inc. + Copyright (C) 2010-2013 Free Software Foundation, Inc. Contributed by Nicola Pero This file is part of GCC. Index: libobjc/acinclude.m4 === --- libobjc/acinclude.m42013-02-02 21:33:00.723873757 + +++ libobjc/acinclude.m42013-02-03 10:31:00.244953931 + @@ -1,5 +1,4 @@ -dnl Copyright 1994, 1995, 1996, 1997, 1998, 1999, 2001, 2010 -dnl Free Software Foundation, Inc. +dnl Copyright (C) 1994-2013 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, dnl with or without modifications, as long as this notice is preserved. Index: libobjc/class.c === --- libobjc/class.c 2013-02-02 21:33:00.728873795 + +++ libobjc/class.c 2013-02-03 10:31:00.251953967 + @@ -1,6 +1,5 @@ /* GNU Objective C Runtime class related functions - Copyright (C) 1993, 1995, 1996, 1997, 2001, 2002, 2009, 2010, 2011 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. Contributed by Kresten Krab Thorup and Dennis Glatting. Lock-free class table code designed and written from scratch by Index: libobjc/configure.ac === --- libobjc/configure.ac2013-02-02 21:33:00.728873795 + +++ libobjc/configure.ac2013-02-03 10:31:00.252953972 + @@ -1,6 +1,5 @@ # Process this file with autoconf to produce a configure script. -# Copyright 1994, 1995, 1996, 1997, 1998, 1999, 2001, 2002, 2003, 2004 -# 2005, 2006, 2009, 2011, 2012 Free Software Foundation, Inc. +# Copyright (C) 1994-2013 Free Software Foundation, Inc. # Originally contributed by Dave Love (d.l...@dl.ac.uk). # #This file is part of GCC. Index: libobjc/encoding.c === --- libobjc/encoding.c 2013-02-02 21:33:00.727873787 + +++ libobjc/encoding.c 2013-02-03 10:31:00.252953972 + @@ -1,6 +1,5 @@ /* Encoding of types for Objective C. - Copyright (C) 1993, 1995, 1996, 1997, 1998, 2000, 2002, 2004, 2009, 2010 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. Contributed by Kresten Krab Thorup Bitfield support by Ovidiu Predescu Index: libobjc/error.c === --- libobjc/error.c
Re: RFA: Fix mips-sde-elf fallout from 4.8 extv/extzv changes
gcc/ * expmed.c (extract_bit_field_1): Pass the full width of the structure to get_best_reg_extraction_insn. OK, thanks. -- Eric Botcazou
RFA: Update copyright for libquadmath
Most of the files here say This file is part of the GNU C Library or the GNU MP Library, so maybe we don't actually want to update them. Please let me know if so. Tested on x86_64-linux-gnu. OK to install? Richard libquadmath/ Update copyright years for libquadmath. Index: libquadmath/math/cacoshq.c === --- libquadmath/math/cacoshq.c 2013-02-02 21:33:00.692873520 + +++ libquadmath/math/cacoshq.c 2013-02-03 10:33:31.345716627 + @@ -1,5 +1,5 @@ /* Return arc hyperbole cosine for __float128 value. - Copyright (C) 1997, 1998, 2006 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/cacosq.c === --- libquadmath/math/cacosq.c 2013-02-02 21:33:00.692873520 + +++ libquadmath/math/cacosq.c 2013-02-03 10:33:31.346716634 + @@ -1,5 +1,5 @@ /* Return cosine of complex __float128 value. - Copyright (C) 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/casinhq.c === --- libquadmath/math/casinhq.c 2013-02-02 21:33:00.694873534 + +++ libquadmath/math/casinhq.c 2013-02-03 10:33:31.35271 + @@ -1,5 +1,5 @@ /* Return arc hyperbole sine for __float128 value. - Copyright (C) 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/casinq.c === --- libquadmath/math/casinq.c 2013-02-02 21:33:00.693873527 + +++ libquadmath/math/casinq.c 2013-02-03 10:33:31.368716756 + @@ -1,5 +1,5 @@ /* Return arc sine of complex __float128 value. - Copyright (C) 1997 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/catanhq.c === --- libquadmath/math/catanhq.c 2013-02-02 21:33:00.693873527 + +++ libquadmath/math/catanhq.c 2013-02-03 10:33:31.369716761 + @@ -1,5 +1,5 @@ /* Return arc hyperbole tangent for __float128 value. - Copyright (C) 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/catanq.c === --- libquadmath/math/catanq.c 2013-02-02 21:33:00.689873497 + +++ libquadmath/math/catanq.c 2013-02-03 10:33:31.376716800 + @@ -1,5 +1,5 @@ /* Return arc tangent of complex __float128 value. - Copyright (C) 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/ccoshq.c === --- libquadmath/math/ccoshq.c 2013-02-02 21:33:00.691873512 + +++ libquadmath/math/ccoshq.c 2013-02-03 10:33:31.376716800 + @@ -1,5 +1,5 @@ /* Complex cosine hyperbole function for complex __float128. - Copyright (C) 1997-2012 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/cexpq.c === --- libquadmath/math/cexpq.c2013-02-02 21:33:00.691873512 + +++ libquadmath/math/cexpq.c2013-02-03 10:33:31.377716805 + @@ -1,5 +1,5 @@ /* Return value of complex exponential function for complex __float128 value. - Copyright (C) 1997-2012 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper drep...@cygnus.com, 1997. Index: libquadmath/math/cimagq.c === --- libquadmath/math/cimagq.c 2013-02-02 21:33:00.693873527 + +++ libquadmath/math/cimagq.c 2013-02-03 10:33:31.377716805 + @@ -1,5 +1,5 @@ /* Return imaginary part of complex __float128 value. - Copyright (C) 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1997-2013 Free Software Foundation, Inc. This file is
[committed] Fix copyright typo
Committed as obvious. Richard libstdc++-v3/ * config/os/mingw32/ctype_inline.h: Fix copyright typo. Index: libstdc++-v3/config/os/mingw32/ctype_inline.h === --- libstdc++-v3/config/os/mingw32/ctype_inline.h 2013-01-06 11:17:47.0 + +++ libstdc++-v3/config/os/mingw32/ctype_inline.h 2013-02-02 10:06:22.706943134 + @@ -1,6 +1,6 @@ // Locale support -*- C++ -*- -// Copyright (C) 2000, 2009, 2010,2 011 Free Software Foundation, Inc. +// Copyright (C) 2000, 2009, 2010, 2011 Free Software Foundation, Inc. // // This file is part of the GNU ISO C++ Library. This library is free // software; you can redistribute it and/or modify it under the
Re: RFA: Update copyright for libobjc
OK. Thanks On 3 Feb 2013, at 10:33, Richard Sandiford wrote: Update the copyright notices for libobjc. Tested on x86_64-linux-gnu. OK to install? Richard libbjc/ Update copyright years. Index: libobjc/Makefile.in === --- libobjc/Makefile.in 2013-02-02 21:33:00.727873787 + +++ libobjc/Makefile.in 2013-02-03 10:31:00.272954078 + @@ -1,6 +1,5 @@ # Makefile for GNU Objective C runtime library. -# Copyright 1993, 1995, 1996, 1997, 1998, 1999, 2001, 2002, 2003, 2004, -# 2005, 2006, 2007, 2008, 2009, 2010, 2011 Free Software Foundation, Inc. +# Copyright (C) 1993-2013 Free Software Foundation, Inc. #This file is part of GCC. Index: libobjc/NXConstStr.m === --- libobjc/NXConstStr.m 2013-02-02 21:33:00.724873765 + +++ libobjc/NXConstStr.m 2013-02-03 10:31:00.225953831 + @@ -1,5 +1,5 @@ /* Implementation of the NXConstantString class for Objective-C. - Copyright (C) 1995, 2009, 2010 Free Software Foundation, Inc. + Copyright (C) 1995-2013 Free Software Foundation, Inc. Contributed by Pieter J. Schoenmakers ti...@es.ele.tue.nl This file is part of GCC. Index: libobjc/Object.m === --- libobjc/Object.m 2013-02-02 21:33:00.728873795 + +++ libobjc/Object.m 2013-02-03 10:31:00.226953836 + @@ -1,6 +1,5 @@ /* The implementation of class Object for Objective-C. - Copyright (C) 1993, 1994, 1995, 1997, 2002, 2009, 2010, 2011 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. This file is part of GCC. Index: libobjc/Protocol.m === --- libobjc/Protocol.m2013-02-02 21:33:00.724873765 + +++ libobjc/Protocol.m2013-02-03 10:31:00.243953925 + @@ -1,5 +1,5 @@ /* This file contains the implementation of class Protocol. - Copyright (C) 1993, 2004, 2009, 2010, 2011 Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. This file is part of GCC. Index: libobjc/accessors.m === --- libobjc/accessors.m 2013-02-02 21:33:00.725873772 + +++ libobjc/accessors.m 2013-02-03 10:31:00.272954078 + @@ -1,5 +1,5 @@ /* GNU Objective C Runtime accessors functions - Copyright (C) 2010 Free Software Foundation, Inc. + Copyright (C) 2010-2013 Free Software Foundation, Inc. Contributed by Nicola Pero This file is part of GCC. Index: libobjc/acinclude.m4 === --- libobjc/acinclude.m4 2013-02-02 21:33:00.723873757 + +++ libobjc/acinclude.m4 2013-02-03 10:31:00.244953931 + @@ -1,5 +1,4 @@ -dnl Copyright 1994, 1995, 1996, 1997, 1998, 1999, 2001, 2010 -dnl Free Software Foundation, Inc. +dnl Copyright (C) 1994-2013 Free Software Foundation, Inc. dnl This file is free software; the Free Software Foundation dnl gives unlimited permission to copy and/or distribute it, dnl with or without modifications, as long as this notice is preserved. Index: libobjc/class.c === --- libobjc/class.c 2013-02-02 21:33:00.728873795 + +++ libobjc/class.c 2013-02-03 10:31:00.251953967 + @@ -1,6 +1,5 @@ /* GNU Objective C Runtime class related functions - Copyright (C) 1993, 1995, 1996, 1997, 2001, 2002, 2009, 2010, 2011 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. Contributed by Kresten Krab Thorup and Dennis Glatting. Lock-free class table code designed and written from scratch by Index: libobjc/configure.ac === --- libobjc/configure.ac 2013-02-02 21:33:00.728873795 + +++ libobjc/configure.ac 2013-02-03 10:31:00.252953972 + @@ -1,6 +1,5 @@ # Process this file with autoconf to produce a configure script. -# Copyright 1994, 1995, 1996, 1997, 1998, 1999, 2001, 2002, 2003, 2004 -# 2005, 2006, 2009, 2011, 2012 Free Software Foundation, Inc. +# Copyright (C) 1994-2013 Free Software Foundation, Inc. # Originally contributed by Dave Love (d.l...@dl.ac.uk). # #This file is part of GCC. Index: libobjc/encoding.c === --- libobjc/encoding.c2013-02-02 21:33:00.727873787 + +++ libobjc/encoding.c2013-02-03 10:31:00.252953972 + @@ -1,6 +1,5 @@ /* Encoding of types for Objective C. - Copyright (C) 1993, 1995, 1996, 1997, 1998, 2000, 2002, 2004, 2009, 2010 - Free Software Foundation, Inc. + Copyright (C) 1993-2013 Free Software Foundation, Inc. Contributed by Kresten Krab
[Ada] Issue sorry message for Scalar_Storage_Order
The support for the new attribute Scalar_Storage_Order isn't complete yet so
this patch adds a couple of calls to sorry for it in gigi.
Tested on x86_64-suse-linux, applied on the mainline.
2013-02-03 Eric Botcazou ebotca...@adacore.com
* gcc-interface/decl.c: Include diagnostic-core.h.
(gnat_to_gnu_entity) E_Array_Type: Sorry if Reverse_Storage_Order
is set on the entity.
E_Record_Type: Likewise.
* gcc-interface/Make-lang.in (ada/decl.o): Add $(DIAGNOSTIC_CORE_H).
--
Eric BotcazouIndex: gcc-interface/decl.c
===
--- gcc-interface/decl.c (revision 195690)
+++ gcc-interface/decl.c (working copy)
@@ -33,6 +33,7 @@
#include ggc.h
#include target.h
#include tree-inline.h
+#include diagnostic-core.h
#include ada.h
#include types.h
@@ -2244,6 +2245,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entit
for (index = ndim - 1; index = 0; index--)
{
tem = build_nonshared_array_type (tem, gnu_index_types[index]);
+ if (Reverse_Storage_Order (gnat_entity))
+ sorry (non-default Scalar_Storage_Order);
TYPE_MULTI_ARRAY_P (tem) = (index 0);
if (array_type_has_nonaliased_component (tem, gnat_entity))
TYPE_NONALIASED_COMPONENT (tem) = 1;
@@ -2957,6 +2960,8 @@ gnat_to_gnu_entity (Entity_Id gnat_entit
gnu_type = make_node (tree_code_for_record_type (gnat_entity));
TYPE_NAME (gnu_type) = gnu_entity_name;
TYPE_PACKED (gnu_type) = (packed != 0) || has_rep;
+ if (Reverse_Storage_Order (gnat_entity))
+ sorry (non-default Scalar_Storage_Order);
if (!definition)
{
Index: gcc-interface/Make-lang.in
===
--- gcc-interface/Make-lang.in (revision 195690)
+++ gcc-interface/Make-lang.in (working copy)
@@ -986,8 +986,8 @@ ada/cuintp.o : ada/gcc-interface/cuintp.
$(COMPILER) -c $(ALL_COMPILERFLAGS) -I.. $(ALL_CPPFLAGS) $ -o $@
ada/decl.o : ada/gcc-interface/decl.c $(CONFIG_H) $(SYSTEM_H) coretypes.h \
- $(TM_H) $(TREE_H) $(FLAGS_H) toplev.h $(TARGET_H) \
- $(TREE_INLINE_H) ada/gcc-interface/ada.h ada/types.h ada/atree.h \
+ $(TM_H) $(TREE_H) $(FLAGS_H) toplev.h $(TARGET_H) $(TREE_INLINE_H) \
+ $(DIAGNOSTIC_CORE_H) ada/gcc-interface/ada.h ada/types.h ada/atree.h \
ada/elists.h ada/namet.h ada/nlists.h ada/repinfo.h ada/snames.h \
ada/stringt.h ada/uintp.h ada/fe.h ada/sinfo.h ada/einfo.h $(ADA_TREE_H) \
ada/gcc-interface/gigi.h gt-ada-decl.h
Re: RFA: Update copyright for libitm
On Sun, Feb 03, 2013 at 10:22:13AM +, Richard Sandiford wrote: Update the copyright notices for libitm. Tested on x86_64-linux-gnu. OK to install? libitm/ Update copyright years Ok, thanks. Jakub
Re: RFA: Update copyright for libmudflap
On Sun, Feb 03, 2013 at 10:28:41AM +, Richard Sandiford wrote: libmudflap/ Update copyright years. Ok. Jakub
Re: RFA: Update copyright for libobjc
On Sun, Feb 03, 2013 at 10:33:26AM +, Richard Sandiford wrote: Update the copyright notices for libobjc. Tested on x86_64-linux-gnu. OK to install? Richard libbjc/ Update copyright years. Ok. Jakub
Re: RFA: Update copyright for libquadmath
On Sun, Feb 03, 2013 at 10:38:05AM +, Richard Sandiford wrote: Most of the files here say This file is part of the GNU C Library or the GNU MP Library, so maybe we don't actually want to update them. Please let me know if so. Tested on x86_64-linux-gnu. OK to install? Richard libquadmath/ Update copyright years for libquadmath. Dunno if we want this, all these files are copied over (with some minor adjustments) from glibc, so I wonder if they shouldn't switch to the range Copyrights only when the glibc origins do. Jakub
Re: RFA: Update copyright for libssp
On Sun, Feb 03, 2013 at 10:39:55AM +, Richard Sandiford wrote: Seems like one of the simpler directories copyrightwise. Tested on x86_64-linux-gnu. OK to install? Richard libssp/ Update copyright years. Similar to libquadmath, these files are copied/modified over from glibc origins. Makefile.am change is of course fine. Jakub
[PATCH] collect2 should ignore AIX shared objects marked F_LOADONLY
Although AIX now supports shared objects with the .so suffix, AIX's native support for shared objects are special object files in archive files. Shared object versioning is supported with multiple shared objects in a single archive file. Shared object files providing earlier versions of symbols normally have the F_LOADONLY flag set in the header and only the most recent version is used for newly linked applications. Andrew Dixie pointed out that GCC collect2 is scanning all versions of shared objects instead of only the ones that are set for new linking. The patch below updates GCC_CHECK_HDR to skip object files marked F_LOADONLY. Bootstrapped on powerpc-ibm-aix7.1.0.0 * collect2.c (GCC_CHECK_HDR): Do not scan objects with F_LOADONLY flag set. Index: collect2.c === --- collect2.c (revision 195687) +++ collect2.c (working copy) @@ -2763,12 +2763,14 @@ /* 0757 = U803XTOCMAGIC (AIX 4.3) and 0767 = U64_TOCMAGIC (AIX V5) */ #if TARGET_AIX_VERSION = 51 # define GCC_CHECK_HDR(X) \ - ((HEADER (X).f_magic == U802TOCMAGIC ! aix64_flag) \ - || (HEADER (X).f_magic == 0767 aix64_flag)) + (((HEADER (X).f_magic == U802TOCMAGIC ! aix64_flag) \ + || (HEADER (X).f_magic == 0767 aix64_flag)) \ + !(HEADER (X).f_flags F_LOADONLY)) #else # define GCC_CHECK_HDR(X) \ - ((HEADER (X).f_magic == U802TOCMAGIC ! aix64_flag) \ - || (HEADER (X).f_magic == 0757 aix64_flag)) + (((HEADER (X).f_magic == U802TOCMAGIC ! aix64_flag) \ + || (HEADER (X).f_magic == 0757 aix64_flag)) \ + !(HEADER (X).f_flags F_LOADONLY)) #endif #endif
Re: RFA: Update copyright in libstdc++-v3
Hi, Tested on x86_64-linux-gnu, including with a -D_GLIBCXX_DEBUG run. OK to install? Ok, thanks! Paolo
Re: Ping: unreviewed copyright patches
On 02/03/13 02:19, Richard Sandiford wrote: Not exactly the most exciting patches, and certainly not worth more than one ping, but: fixincludes copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00442.html You missed the file header. Why bother with dual update issues? --- mkheaders.in(revision 194742) +++ mkheaders.in(working copy) @@ -1,6 +1,6 @@ #!/bin/sh -# Copyright (C) 2002, 2007, 2009 Free Software Foundation, Inc. +# GCC Copyright (C) 2002-2013 Free Software Foundation, Inc. #This file is part of GCC. @@ -41,10 +41,9 @@ if [ x$1 = x--version ] ; then echo mkheaders (GCC) version $version -echo Copyright 2002, 2007, 2009 Free Software Foundation, Inc. -echo This program is free software; you may redistribute it under the -echo terms of the GNU General Public License. This program has -echo absolutely no warranty. +sed '\@http://www.gnu.org/licenses@q + \@/bin/sh@d + s/^#* *//' `which $0` exit 0 fi Alternatively (and probably better): --- mkheaders.in(revision 194742) +++ mkheaders.in(working copy) @@ -1,22 +1,22 @@ #!/bin/sh +copyright=' +GCC Copyright (C) 2002-2013 Free Software Foundation, Inc. -# Copyright (C) 2002, 2007, 2009 Free Software Foundation, Inc. +This file is part of GCC. -#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/. +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/.' # Basic information target=@target@ @@ -41,10 +41,7 @@ if [ x$1 = x--version ] ; then echo mkheaders (GCC) version $version -echo Copyright 2002, 2007, 2009 Free Software Foundation, Inc. -echo This program is free software; you may redistribute it under the -echo terms of the GNU General Public License. This program has -echo absolutely no warranty. +echo $copyright exit 0 fi Either way === http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00443.html Is the project fixincludes, inclhack, gcc or unnamed? I think if you go to the trouble to clean all this up, one should be picked and it should be made consistent. :) Personally, I think GCC first, fixincludes second. Index: gcc/fixincludes/fixinc.in === --- gcc.orig/fixincludes/fixinc.in +++ gcc/fixincludes/fixinc.in @@ -6,8 +6,7 @@ # # See README-fixinc for more information. # -# fixincludes copyright (c) 1998, 1999, 2000, 2002, 2009 -# The Free Software Foundation, Inc. +# fixincludes copyright (C) 1998-2013 Free Software Foundation, Inc. # #
Re: Ping: unreviewed copyright patches
Bruce Korb bk...@gnu.org writes: On 02/03/13 02:19, Richard Sandiford wrote: Not exactly the most exciting patches, and certainly not worth more than one ping, but: fixincludes copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00442.html You missed the file header. That was deliberately in patch 2 though. Why bother with dual update issues? Well, the point is that patch 2 is scripted. Running the script as-is would update both the header and version output. That's actually what I did in the original patch I posted back in January. However, Joseph pointed out that mkheaders should only print the last year rather than a list of years or a range of years (apparently that's GNU policy): http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00320.html so patch 1 was supposed to be changing that first. In other words, the copyright= suggestion seemed to be about having the same string in both the headers and output, which is what you also get by running the script. But apparently the copyright notice in the file header and the copyright notice that we print are supposed to be different. http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00443.html Is the project fixincludes, inclhack, gcc or unnamed? I think if you go to the trouble to clean all this up, one should be picked and it should be made consistent. :) OK, fair enough. I guess I'll withdraw the fixincludes bits. Believe me, I don't find copyright year lists vs. ranges any more interesting than the next man. This really was supposed to be an attempt to make things simpler. :-) Thanks, Richard
Re: Ping: unreviewed copyright patches
On 02/03/13 09:42, Richard Sandiford wrote: You missed the file header. That was deliberately in patch 2 though. OK. Why bother with dual update issues? Well, the point is that patch 2 is scripted. OK: echo $copyright | sed 's/(C) 2002-/(C) /' and now you print the right date and never have to hand update again. And if the notice gets changed because of a new and better address or whatever, just one place and not two. I hate hand updating mind numbing stuff like this. As you say: Believe me, I don't find copyright year lists vs. ranges any more interesting than the next man. This really was supposed to be an attempt to make things simpler. simpler as in trying to never have to think about it any more? :)
[PATCH][revised] fix PR sanitizer/55617
Currently darwin is unable to utilize libasan with constructors due to the
lack of
constructor priority support on that target. The asan_finish_file routine
inserts an
essential __asan_init into the array of constructors (via the __mod_init_func
section).
However the insertion occurs at the end, and due to the lack of priority
support for
constructors, these are executed from the start of the array of constructors on
program
startup. This causes code any instrumented code that executes before the
__asan_init
call to crash.
Since darwin sets...
#undef SUPPORTS_INIT_PRIORITY
#define SUPPORTS_INIT_PRIORITY 0
in gcc/config/darwin.h, all constructors are automatically set to
#define DEFAULT_INIT_PRIORITY 65535
in gcc/collect2.c. Any code that attempts to set the constructor/destructor
priority
on darwin results in a compile time error of constructor priorities are not
supported.
So asan alone should be unique in emitting priorities different from 65535 on
darwin.
The attached patch uses a va_gc vector of constructor symbol/priority records
to queue
this data as it is generated in calls to machopic_asm_out_constructor. Any
instances of
the static constructor with priority MAX_RESERVED_INIT_PRIORITY-1 emitted by
asan are
inserted safely in the front of the vector queue which retains the original
order of the
remaining constructors in the queue. The contents of the vector queue are later
processed
in a new finalize_ctors routine called from darwin_file_end if necessary. The
patch also
adds a g++.dg/asan/pr55617.C test case which is targeted to i?86-*-darwin* and
x86_64-*-darwin*.
The patch reduces the failures observed when running
make -k check-g++ RUNTESTFLAGS=--target_board=unix'{-fsanitize=address}'
from 323 to only 85 on darwin (similar to the results on linux). The cov.C
testcase also
fails on gcc trunk with -fsanitize=address when recrafted into a dynamic shared
library
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55617#c28. This patch eliminates
those
crashes. This problem doesn't extend to when the shared library or module is
dlopen'd
(which works in stock gcc trunk and with this patch as well).
The patch has been bootstrap and regression tested on x86_64-apple-darwin12.
Okay for gcc trunk?
Jack
ps Unfortunately the flag_sort variable is unavailable inside of
machopic_asm_out_constructor
so we have to unconditionally test for priority == MAX_RESERVED_INIT_PRIORITY -
1.
/gcc
2013-02-03 Alexander Potapenko gli...@google.com
Jack Howarth howa...@bromo.med.uc.edu
PR sanitizer/55617
* config/darwin.c (machopic_asm_out_constructor): Use vector to
queue constructors while inserting asan static constructors at front.
(finalize_ctors): New routine to output queued constructors.
(darwin_file_end): Use finalize_ctors.
/gcc/testsuite
2013-02-03 Alexander Potapenko gli...@google.com
Jack Howarth howa...@bromo.med.uc.edu
PR sanitizer/55617
* g++.dg/asan/pr55617.C: New test.
Index: gcc/config/darwin.c
===
--- gcc/config/darwin.c (revision 195701)
+++ gcc/config/darwin.c (working copy)
@@ -83,6 +83,13 @@ along with GCC; see the file COPYING3.
kernel) the stubs might still be required, and this will be set true. */
int darwin_emit_branch_islands = false;
+typedef struct GTY(()) ctor_record {
+ rtx symbol;
+ int priority;/* constructor priority */
+} ctor_record;
+
+static GTY(()) vecctor_record, va_gc *ctors = NULL;
+
/* A flag to determine whether we are running c++ or obj-c++. This has to be
settable from non-c-family contexts too (i.e. we can't use the c_dialect_
functions). */
@@ -1708,15 +1715,33 @@ machopic_select_rtx_section (enum machin
void
machopic_asm_out_constructor (rtx symbol, int priority ATTRIBUTE_UNUSED)
{
+ ctor_record new_elt = {symbol, priority};
+ /* Force asan static constructors to front of constructor queue. */
+ if (priority == MAX_RESERVED_INIT_PRIORITY - 1)
+vec_safe_insert(ctors, 0, new_elt);
+ else
+vec_safe_push (ctors, new_elt);
+
+ if (! MACHOPIC_INDIRECT)
+fprintf (asm_out_file, .reference .constructors_used\n);
+}
+
+static void
+finalize_ctors()
+{
+ unsigned int i;
+ ctor_record *elt;
+
if (MACHOPIC_INDIRECT)
switch_to_section (darwin_sections[mod_init_section]);
else
switch_to_section (darwin_sections[constructor_section]);
- assemble_align (POINTER_SIZE);
- assemble_integer (symbol, POINTER_SIZE / BITS_PER_UNIT, POINTER_SIZE, 1);
- if (! MACHOPIC_INDIRECT)
-fprintf (asm_out_file, .reference .constructors_used\n);
+ FOR_EACH_VEC_SAFE_ELT (ctors, i, elt)
+{
+ assemble_align (POINTER_SIZE);
+ assemble_integer (elt-symbol, POINTER_SIZE / BITS_PER_UNIT,
POINTER_SIZE, 1);
+}
}
void
@@ -2762,6 +2787,8 @@ darwin_file_start (void)
void
darwin_file_end (void)
{
+ if
[committed] Delete GO_IF_LEGITIMATE_ADDRESS on PA target
The main goal of this change was to replace the deprecated GO_IF_LEGITIMATE_ADDRESS macro with TARGET_LEGITIMATE_ADDRESS_P. The patch also modifies the LEGITIMIZE_RELOAD_ADDRESS macro to use pa_legitimize_reload_address and tightens the memory address constraint used for floating point moves and stores. While moving this code was straight forward, as I tried to simplify the handling of REG+SMALLINT and symbolic addresses, bugs started crawling out of the woodwork. I discovered that symbolic_memory_operand was broken. There were reload issues for loading constants to floating point registers. Some patterns in pa.md just made matters worse by creating RTL with constant sources. The DF and SFmode move expanders had hacks intended to reject these constants but they aren't supposed to fail. So, in the end, I touched a lot more code than I expected. The tricky part of pa_legitimate_address_p is the forcing of secondary reloads to fix SImode and DImode floating point instructions. In the end, I decided reload_in_progress strict was the correct way to do this. strict by itself fails to handle asm's correctly. Tested on hppa-unknown-linux-gnu, hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11 with no regressions. Committed to trunk. Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
[SH] PR 55146 - Add testcase
Hi,
The attached patch adds the testcase as it was originally reported in
the PR. Although it's not a problem on 4.8, I think it would be better
to have this in the test suite.
Tested with
make -k check-gcc RUNTESTFLAGS=sh.exp=pr55146.c --target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
OK for trunk?
Cheers,
Oleg
testsuite/ChangeLog:
PR target/55146:
* gcc.target/sh/pr55146.c: New.
Index: gcc/testsuite/gcc.target/sh/pr55146.c
===
--- gcc/testsuite/gcc.target/sh/pr55146.c (revision 0)
+++ gcc/testsuite/gcc.target/sh/pr55146.c (revision 0)
@@ -0,0 +1,50 @@
+/* Check that the 'extu.b' instruction is generated for short jump tables. */
+/* { dg-do compile { target sh*-*-* } } */
+/* { dg-options -Os } */
+/* { dg-skip-if { sh*-*-* } { -m5*} { } } */
+/* { dg-final { scan-assembler extu.b } } */
+
+int
+test (int arg)
+{
+ int rc;
+ switch (arg)
+{
+case 0:
+ asm (nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ mov r4,%0
+ : =r (rc)
+ : r (arg));
+ break;
+case 1:
+ asm (nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ mov r5,%0
+ : =r (rc)
+ : r (arg));
+ break;
+case 2:
+ asm (nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ mov r6,%0
+ : =r (rc)
+ : r (arg));
+ break;
+case 3:
+ asm (nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ mov r7,%0
+ : =r (rc)
+ : r (arg));
+ break;
+case 4:
+ asm (nop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ nop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\tnop\n\t
+ mov r8,%0
+ : =r (rc)
+ : r (arg));
+ break;
+}
+ return rc;
+}
Re: Ping: unreviewed copyright patches
On Sun, Feb 03, 2013 at 10:19:47AM +, Richard Sandiford wrote: libdecnumber copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00647.html I've refreshed my memory on the use of year ranges in the copyright notice (http://www.gnu.org/licenses/gpl-howto.html). Looking at the ChangeLog, I think it's reasonable to assert copyright for 2005-2011, but not 2012 or 2013. I am happy with the change if the copyright is updated from (firstyear-for-the-file) to 2011. Cheers, Ben signature.asc Description: Digital signature
Re: hasher speed traits
On 2 February 2013 21:57, François Dumont wrote: Hi Here is the last patch I can think of for 4.8. Thanks to it default performance reported in performance/23_containers/insert/54075.cc and performance/23_containers/insert_erase/41975.cc are always the best: Excellent. Ok to commit ? Yes, thanks for all your work on this. As Benjamin suggested, we should think about which changes are user-visible and worth documenting in the GCC 4.8 release notes. We should certainly note somewhere that it's important for the hash function to be noexcept to avoid decreased performance.
[Patch, fortran] PR54107: ICE on recursive interfaces and PR54195: symbol bogusly inserted twice in the interface.
Hello,
The following patches fix both PR54107 and PR54195.
- In PR54107(comment 26), the procedure result is a procedure pointer
whose interface is the procedure itself, which leads to an infinite
recursion during resolution.
- In PR54195, a type's type bound procedures are resolved twice, leading
to a symbol being added twice in an interface and rejected.
The fix, as discussed in PR54195, adds a flag to mark a symbol as
resolved. This leads to two regressions. For class_20, a check to skip
result symbols had to be removed (which was there to avoid duplicated
resolution). For initialization_27 (among a few others) the code adding
the default initialization code was guarded by a check against
gfc_current_ns, which always ended triggering when there was more than
one resolution but may not anymore. The fix removes it; I checked that
gfc_current_ns wasn't used in the following code.
The second fix makes the recursion through resolve_symbol, so that the
flag just added triggers and PR54195 is fixed.
Regression tested on x86_64-unknown-linux-gnu. OK for trunk?
Mikael
2013-02-03 Mikael Morin mik...@gcc.gnu.org
PR fortran/54107
PR fortran/54195
* gfortran.h (struct symbol_attribute): New field 'resolved'.
* resolve.c (resolve_fl_var_and_proc): Don't skip result symbols.
(resolve_symbol): Skip duplicate calls. Don' check the current
namespace.
2013-02-03 Mikael Morin mik...@gcc.gnu.org
PR fortran/54107
* gfortran.dg/recursive_interface_1.f90: New test.
diff --git a/gfortran.h b/gfortran.h
index 16751b4..af2b45a 100644
--- a/gfortran.h
+++ b/gfortran.h
@@ -810,6 +810,9 @@ typedef struct
/* Attributes set by compiler extensions (!GCC$ ATTRIBUTES). */
unsigned ext_attr:EXT_ATTR_NUM;
+ /* Used to avoid multiple resolutions of a single symbol. */
+ unsigned resolved:1;
+
/* The namespace where the attribute has been set. */
struct gfc_namespace *volatile_ns, *asynchronous_ns;
}
diff --git a/resolve.c b/resolve.c
index d6bae43..3b74c6f 100644
--- a/resolve.c
+++ b/resolve.c
@@ -11051,11 +11051,6 @@ resolve_fl_var_and_proc (gfc_symbol *sym, int mp_flag)
{
gfc_array_spec *as;
- /* Avoid double diagnostics for function result symbols. */
- if ((sym-result || sym-attr.result) !sym-attr.dummy
- (sym-ns != gfc_current_ns))
-return SUCCESS;
-
if (sym-ts.type == BT_CLASS sym-attr.class_ok)
as = CLASS_DATA (sym)-as;
else
@@ -13170,6 +13165,10 @@ resolve_symbol (gfc_symbol *sym)
gfc_array_spec *as;
bool saved_specification_expr;
+ if (sym-attr.resolved)
+return;
+ sym-attr.resolved = 1;
+
if (sym-attr.artificial)
return;
@@ -13779,7 +13778,6 @@ resolve_symbol (gfc_symbol *sym)
described in 14.7.5, to those variables that have not already
been assigned one. */
if (sym-ts.type == BT_DERIVED
- sym-ns == gfc_current_ns
!sym-value
!sym-attr.allocatable
!sym-attr.alloc_comp)
! { dg-do compile }
!
! PR fortran/54107
! The compiler used to ICE on recursive interfaces.
module m
contains
function foo() result(r1)
procedure(foo), pointer :: r1
end function foo
function bar() result(r2)
procedure(baz), pointer :: r2
end function bar
function baz() result(r3)
procedure(bar), pointer :: r3
end function baz
end module m
2013-02-03 Mikael Morin mik...@gcc.gnu.org
PR fortran/54195
* resolve.c (resolve_typebound_procedures): Recurse through
resolve_symbol.
2013-02-03 Mikael Morin mik...@gcc.gnu.org
PR fortran/54195
* gfortran.dg/defined_assignment_4.f90: New test.
* gfortran.dg/defined_assignment_5.f90: New test.
diff --git a/resolve.c b/resolve.c
index 3b74c6f..6bec662 100644
--- a/resolve.c
+++ b/resolve.c
@@ -12344,7 +12344,7 @@ resolve_typebound_procedures (gfc_symbol* derived)
super_type = gfc_get_derived_super_type (derived);
if (super_type)
-resolve_typebound_procedures (super_type);
+resolve_symbol (super_type);
resolve_bindings_derived = derived;
resolve_bindings_result = SUCCESS;
! { dg-do compile }
!
! PR fortran/54195
! The compiler used to diagnose a duplicate entity in the assignment interface
! because NC was resolved twice.
!
! Contributed by Damian Rouson dam...@rouson.net
module import_clashes_with_generic
type ,abstract :: foo
contains
procedure :: unary
generic :: operator(-) = unary
end type
abstract interface
integer function bar()
import :: foo
end function
end interface
contains
integer function unary(rhs)
class(foo) ,intent(in) :: rhs
end function
end module
! { dg-do compile }
!
! PR fortran/54195
! The compiler used to diagnose a duplicate entity in the assignment interface
! because NC was resolved twice.
!
! Contributed by Andrew Benson aben...@obs.carnegiescience.edu
module gn
implicit none
type :: nc
contains
procedure :: assign = nca
Re: [SH] PR 55146 - Add testcase
Oleg Endo oleg.e...@t-online.de wrote:
The attached patch adds the testcase as it was originally reported in
the PR. Although it's not a problem on 4.8, I think it would be better
to have this in the test suite.
Tested with
make -k check-gcc RUNTESTFLAGS=sh.exp=pr55146.c --target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
OK for trunk?
OK.
Regards,
kaz
Re: [SH] PR 54386 - Add testcase
Oleg Endo oleg.e...@t-online.de wrote:
PR 54386 has been fixed a while ago already and it wasn't really an SH
specific problem. Still, I'd like to add an SH specific test case for
this.
Tested with
make -k check-gcc RUNTESTFLAGS=sh.exp=pr54386.c --target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}
OK for trunk?
OK.
Regards,
kaz
Re: Ping: unreviewed copyright patches
On Sun, Feb 3, 2013 at 2:19 AM, Richard Sandiford rdsandif...@googlemail.com wrote: Not exactly the most exciting patches, and certainly not worth more than one ping, but: libgcc copyright http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00642.html This is OK. Thanks. Don't these count as obvious at this point? Ian
