Bug 100779 - Partial specialization with non-deducible template parameters accepted
Summary: Partial specialization with non-deducible template parameters accepted
Status: UNCONFIRMED
Alias: None
Product: gcc
Classification: Unclassified
Component: c++ (show other bugs)
Version: 11.0
: P3 normal
Target Milestone: ---
Assignee: Not yet assigned to anyone
URL:
Keywords: accepts-invalid
Depends on:
Blocks:
 
Reported: 2021-05-26 19:08 UTC by Patrick Palka
Modified: 2023-06-26 19:36 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Palka 2021-05-26 19:08:10 UTC
template <int> struct A;
template <class> struct B;
template <class T> struct B<A<sizeof(T)>> { };

This partial specialization should be rejected because its template parameter T is not deducible.
Comment 1 Patrick Palka 2021-05-26 19:15:59 UTC
Another accepts-invalid example:

template <int> struct A;
template <class> struct B;
template <int N> struct B<A<N+0>> { };
Comment 2 Patrick Palka 2021-05-26 23:49:11 UTC
The problem is ultimately that for_each_template_parm doesn't honor included_nondeduced_p correctly.
Comment 3 GCC Commits 2021-05-27 16:37:24 UTC
The master branch has been updated by Patrick Palka <ppalka@gcc.gnu.org>:

https://gcc.gnu.org/g:9b94785dedb08b006419bec1a402614d9241317a

commit r12-1093-g9b94785dedb08b006419bec1a402614d9241317a
Author: Patrick Palka <ppalka@redhat.com>
Date:   Thu May 27 12:36:39 2021 -0400

    c++: Relax rule for non-type arguments in partial specs [CWG1315]
    
    This implements the wording changes of CWG 1315, which permits non-type
    template arguments in a partial specialization to use template
    parameters more freely.  Delightfully, it seems the only change needed
    is to remove a single check from process_partial_specialization (amidst
    a bunch of whitespace changes).
    
    But that change alone revealed a latent problem with for_each_template_parm:
    it ends up looking into some non-deduced contexts even when
    include_nondeduced_p is false.  This causes us to silently accept some
    partial specializations within the testsuite that contain non-deducible
    non-type template parameters (and that were previously rejected due to
    the rule that CWG 1315 relaxed).  For now this patch makes a minimal
    amount of changes to for_each_template_parm_r so that we continue to
    reject existing ill-formed partial specializations within the testsuite.
    I opened PR c++/100779 to track this issue.
    
            DR 1315
            PR c++/67593
            PR c++/96555
    
    gcc/cp/ChangeLog:
    
            * pt.c (process_partial_specialization): Don't error on a
            non-simple non-type template argument that involves template
            parameters.
            (for_each_template_parm_r): Don't walk TRAIT_EXPR, PLUS_EXPR,
            MULT_EXPR, or SCOPE_REF when include_nondeduced_p is false.
    
    gcc/testsuite/ChangeLog:
    
            * g++.dg/template/partial16.C: New test.
            * g++.dg/template/partial17.C: New test.
            * g++.dg/template/partial18.C: New test.
            * g++.dg/template/partial19.C: New test.
            * g++.dg/cpp0x/pr68724.C: Adjust expected diagnostic for
            ill-formed partial specialization.
            * g++.dg/cpp0x/variadic38.C: Likewise.
            * g++.dg/cpp1z/pr81016.C: Likewise.
            * g++.dg/template/partial5.C: Likewise.
            * g++.old-deja/g++.pt/spec21.C: Likewise.