diff options
| author | Paolo Carlini <paolo.carlini@oracle.com> | 2013-10-25 16:26:10 +0000 |
|---|---|---|
| committer | Paolo Carlini <paolo@gcc.gnu.org> | 2013-10-25 16:26:10 +0000 |
| commit | b02ef56d8d792059f39bef1d9b853d9a2af0aa3b (patch) | |
| tree | 5df10956abfcc99ccbea7d543e1f2f6ec16fd939 | |
| parent | 347593705483c3f425942501ae1a20485fa66ac6 (diff) | |
| download | gcc-b02ef56d8d792059f39bef1d9b853d9a2af0aa3b.zip gcc-b02ef56d8d792059f39bef1d9b853d9a2af0aa3b.tar.gz gcc-b02ef56d8d792059f39bef1d9b853d9a2af0aa3b.tar.bz2 | |
re PR c++/58878 (Template parameter name can be hidden in a template member function defined inside the class specifier)
/cp
2013-10-25 Paolo Carlini <paolo.carlini@oracle.com>
PR c++/58878
* pt.c (check_template_shadow): Don't skip declarations in inline
member templates.
/testsuite
2013-10-25 Paolo Carlini <paolo.carlini@oracle.com>
PR c++/58878
* g++.dg/template/pr58878.C: New.
From-SVN: r204071
| -rw-r--r-- | gcc/cp/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/cp/pt.c | 3 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/template/pr58878.C | 61 |
4 files changed, 74 insertions, 1 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index dfe4a3f..297d015 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2013-10-25 Paolo Carlini <paolo.carlini@oracle.com> + + PR c++/58878 + * pt.c (check_template_shadow): Don't skip declarations in inline + member templates. + 2013-10-25 Tobias Burnus <burnus@net-b.de> PR other/33426 diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index cbb0339a..224be8b 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -3511,7 +3511,8 @@ check_template_shadow (tree decl) name inside a class. We check TPFI to avoid duplicate errors for inline member templates. */ if (decl == olddecl - || TEMPLATE_PARMS_FOR_INLINE (current_template_parms)) + || (DECL_TEMPLATE_PARM_P (decl) + && TEMPLATE_PARMS_FOR_INLINE (current_template_parms))) return true; error ("declaration of %q+#D", decl); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 044b76b..ac5ab6d 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-10-25 Paolo Carlini <paolo.carlini@oracle.com> + + PR c++/58878 + * g++.dg/template/pr58878.C: New. + 2013-10-25 Marc Glisse <marc.glisse@inria.fr> * gcc.dg/tree-ssa/alias-23.c: New file. diff --git a/gcc/testsuite/g++.dg/template/pr58878.C b/gcc/testsuite/g++.dg/template/pr58878.C new file mode 100644 index 0000000..adad9fe --- /dev/null +++ b/gcc/testsuite/g++.dg/template/pr58878.C @@ -0,0 +1,61 @@ +// PR c++/58878 + +// Template-members of non-template class +struct A +{ + template <typename t> // { dg-error "shadows" } + void f() + { + int t = 1; // { dg-error "declaration" } + } + + template <typename t> + void g(); +}; + +template <typename t> // { dg-error "shadows" } +void A::g() +{ + int t = 2; // { dg-error "declaration" } +} + +// (Non-template) Members of template class +template <typename t> // { dg-error "shadows" } +struct B +{ + void f() + { + int t = 3; // { dg-error "declaration" } + } + + void g(); +}; + +template <typename t> // { dg-error "shadows" } +void B<t>::g() +{ + int t = 4; // { dg-error "declaration" } +} + +// Template members of template class +template <typename t> // { dg-error "shadows" } +struct C +{ + template <typename s> // { dg-error "shadows" } + void f() + { + int t = 5; // { dg-error "declaration" } + int s = 6; // { dg-error "declaration" } + } + + template <typename s> + void g(); +}; + +template <typename t> // { dg-error "shadows" } +template <typename s> // { dg-error "shadows" } +void C<t>::g() +{ + int t = 7; // { dg-error "declaration" } + int s = 8; // { dg-error "declaration" } +} |