Re: [PATCH 2/2] c++: Extend PR96204 fix to variable templates

2021-06-30 Thread Patrick Palka via Gcc-patches
On Wed, 30 Jun 2021, Jason Merrill wrote: > On 6/30/21 10:48 AM, Patrick Palka wrote: > > On Tue, 29 Jun 2021, Jason Merrill wrote: > > > > > On 6/29/21 1:57 PM, Patrick Palka wrote: > > > > r12-1829 corrected the access scope during partial specialization > > > > matching of class templates, but

Re: [PATCH 2/2] c++: Extend PR96204 fix to variable templates

2021-06-30 Thread Jason Merrill via Gcc-patches
On 6/30/21 10:48 AM, Patrick Palka wrote: On Tue, 29 Jun 2021, Jason Merrill wrote: On 6/29/21 1:57 PM, Patrick Palka wrote: r12-1829 corrected the access scope during partial specialization matching of class templates, but neglected the variable template case. This patch moves the access scop

Re: [PATCH 2/2] c++: Extend PR96204 fix to variable templates

2021-06-30 Thread Patrick Palka via Gcc-patches
On Tue, 29 Jun 2021, Jason Merrill wrote: > On 6/29/21 1:57 PM, Patrick Palka wrote: > > r12-1829 corrected the access scope during partial specialization > > matching of class templates, but neglected the variable template case. > > This patch moves the access scope adjustment to inside > > most_

Re: [PATCH 2/2] c++: Extend PR96204 fix to variable templates

2021-06-29 Thread Jason Merrill via Gcc-patches
On 6/29/21 1:57 PM, Patrick Palka wrote: r12-1829 corrected the access scope during partial specialization matching of class templates, but neglected the variable template case. This patch moves the access scope adjustment to inside most_specialized_partial_spec, so that all callers can benefit.

[PATCH 2/2] c++: Extend PR96204 fix to variable templates

2021-06-29 Thread Patrick Palka via Gcc-patches
r12-1829 corrected the access scope during partial specialization matching of class templates, but neglected the variable template case. This patch moves the access scope adjustment to inside most_specialized_partial_spec, so that all callers can benefit. Bootstrapped and regtested on x86_64-pc-li