Re: [C++ Patch] for c++/23211

[Fabien Chêne <fabien dot chene at gmail dot com>]

2011/12/27 Jason Merrill <jason@redhat.com>:
> On 12/27/2011 02:12 PM, Fabien Chêne wrote:
>>
>> - ?if (!scope_dependent_p)
>> + ?if (!dependent_scope_p (scope))
>
> I was thinking to change the line
>
>> ?scope_dependent_p = dependent_type_p (scope);
>
> to use dependent_scope_p instead of dependent_type_p.

OK, I have adjusted the patch. It passes regtest as well (on
x86_64_unknown-linux-gnu).

gcc/testsuite/ChangeLog

2011-12-27  Fabien Chêne  <fabien@gcc.gnu.org>

	PR c++/23211
	* g++.dg/template/using18.C: New.
	* g++.dg/template/using19.C: New.
	* g++.dg/template/nested3.C: Remove dg-message at instantiation.
	* g++.dg/template/crash13.C: Likewise.

gcc/cp/ChangeLog

2011-12-27  Fabien Chêne  <fabien@gcc.gnu.org>

	PR c++/23211
	* name-lookup.c (do_class_using_decl): Use dependent_scope_p
	instead of dependent_type_p, to check that a non-dependent
	nested-name-specifier of a class-scope using declaration refers to
	a base, even if the current scope is dependent.
	* parser.c (cp_parser_using_declaration): Set
	USING_DECL_TYPENAME_P to 1 if the DECL is not null. Re-indent a
	'else' close to the prior modification.

-- 
Fabien

Attachment: pr23211.patch
Description: Binary data