diff options
author | Jeffrey Yasskin <jyasskin@google.com> | 2011-07-27 18:38:48 +0000 |
---|---|---|
committer | Jeffrey Yasskin <jyasskin@gcc.gnu.org> | 2011-07-27 18:38:48 +0000 |
commit | ce77c22f1bf5674d60710fba3cf8c43d502db01b (patch) | |
tree | 0046d7d004c0b89fc7974e8ca5f38cc5ebe79a08 | |
parent | f22cfb2e4ef3c8749a4814ebd5d0b39173b05b18 (diff) | |
download | gcc-ce77c22f1bf5674d60710fba3cf8c43d502db01b.zip gcc-ce77c22f1bf5674d60710fba3cf8c43d502db01b.tar.gz gcc-ce77c22f1bf5674d60710fba3cf8c43d502db01b.tar.bz2 |
Propagate the source location of a template's function_decl to the template_decl built out of it.
Propagate the source location of a template's function_decl to the
template_decl built out of it. Without this, specializations
generated before the template was defined get the location of the
closing ')' instead of the location of the template's name, which can
produce inconsistent locations between translation units, which makes
gold's ODR checker unhappy.
* gcc/cp/pt.c (build_template_decl): Copy the function_decl's source
location to the new template_decl.
* gcc/testsuite/g++.old-deja/g++.pt/crash60.C: Updated.
* libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc: Updated.
From-SVN: r176845
-rw-r--r-- | gcc/cp/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/cp/pt.c | 1 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
-rw-r--r-- | gcc/testsuite/g++.old-deja/g++.pt/crash60.C | 4 | ||||
-rw-r--r-- | libstdc++-v3/ChangeLog | 4 | ||||
-rw-r--r-- | libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc | 16 |
6 files changed, 24 insertions, 10 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index d169ed1..653a0ab 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,8 @@ +2011-07-27 Jeffrey Yasskin <jyasskin@google.com> + + * pt.c (build_template_decl): Copy the function_decl's + source location to the new template_decl. + 2011-07-26 Paolo Carlini <paolo.carlini@oracle.com> PR c++/49776 diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 178685c..b9e09af 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -4121,6 +4121,7 @@ build_template_decl (tree decl, tree parms, bool member_template_p) tree tmpl = build_lang_decl (TEMPLATE_DECL, DECL_NAME (decl), NULL_TREE); DECL_TEMPLATE_PARMS (tmpl) = parms; DECL_CONTEXT (tmpl) = DECL_CONTEXT (decl); + DECL_SOURCE_LOCATION (tmpl) = DECL_SOURCE_LOCATION (decl); DECL_MEMBER_TEMPLATE_P (tmpl) = member_template_p; return tmpl; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a5fd846..952c3ca 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2011-07-27 Jeffrey Yasskin <jyasskin@google.com> + + * g++.old-deja/g++.pt/crash60.C: Updated. + 2011-07-27 Jakub Jelinek <jakub@redhat.com> PR target/49866 diff --git a/gcc/testsuite/g++.old-deja/g++.pt/crash60.C b/gcc/testsuite/g++.old-deja/g++.pt/crash60.C index 747af9b..1be4678 100644 --- a/gcc/testsuite/g++.old-deja/g++.pt/crash60.C +++ b/gcc/testsuite/g++.old-deja/g++.pt/crash60.C @@ -5,9 +5,9 @@ // We ICE'd rather than fail to instantiate. template< typename SID, class SDR > -void k( SID sid, SDR* p, +void k( SID sid, SDR* p, // { dg-error "no type named 'T'" } void (SDR::*) - ( typename SID::T ) ); // { dg-error "no type named 'T'" } + ( typename SID::T ) ); struct E { }; struct S { void f( int ); }; diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 38833b7..dd6723f 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,7 @@ +2011-07-27 Jeffrey Yasskin <jyasskin@google.com> + + * testsuite/20_util/weak_ptr/comparison/cmp_neg.cc: Updated. + 2011-07-25 Benjamin Kosnik <bkoz@redhat.com> * include/std/array (at): Remove constexpr when -fno-exceptions. diff --git a/libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc b/libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc index df18712..6eecc2d 100644 --- a/libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc +++ b/libstdc++-v3/testsuite/20_util/weak_ptr/comparison/cmp_neg.cc @@ -44,16 +44,16 @@ main() // { dg-warning "note" "" { target *-*-* } 370 } // { dg-warning "note" "" { target *-*-* } 365 } -// { dg-warning "note" "" { target *-*-* } 357 } +// { dg-warning "note" "" { target *-*-* } 356 } // { dg-warning "note" "" { target *-*-* } 1103 } // { dg-warning "note" "" { target *-*-* } 1098 } -// { dg-warning "note" "" { target *-*-* } 1090 } +// { dg-warning "note" "" { target *-*-* } 1089 } // { dg-warning "note" "" { target *-*-* } 485 } // { dg-warning "note" "" { target *-*-* } 479 } -// { dg-warning "note" "" { target *-*-* } 469 } -// { dg-warning "note" "" { target *-*-* } 814 } -// { dg-warning "note" "" { target *-*-* } 1056 } -// { dg-warning "note" "" { target *-*-* } 1050 } -// { dg-warning "note" "" { target *-*-* } 342 } -// { dg-warning "note" "" { target *-*-* } 292 } +// { dg-warning "note" "" { target *-*-* } 468 } +// { dg-warning "note" "" { target *-*-* } 813 } +// { dg-warning "note" "" { target *-*-* } 1055 } +// { dg-warning "note" "" { target *-*-* } 1049 } +// { dg-warning "note" "" { target *-*-* } 341 } +// { dg-warning "note" "" { target *-*-* } 291 } // { dg-warning "note" "" { target *-*-* } 224 } |