From 6d258f3157f2e035d491929a9b819cae3b24a8e9 Mon Sep 17 00:00:00 2001 From: Jason Merrill Date: Fri, 27 May 2011 15:32:07 -0400 Subject: [PATCH] re PR c++/47687 ([C++0x] Crash on a lambda returning a lambda (using std::function)) PR c++/47687 * pt.c (dependent_type_p_r): Avoid infinite recursion. From-SVN: r174354 --- gcc/cp/ChangeLog | 3 +++ gcc/cp/pt.c | 11 +++++++++-- gcc/testsuite/ChangeLog | 2 ++ gcc/testsuite/g++.dg/cpp0x/lambda/lambda-nested4.C | 9 +++++++++ 4 files changed, 23 insertions(+), 2 deletions(-) create mode 100644 gcc/testsuite/g++.dg/cpp0x/lambda/lambda-nested4.C diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 19ef7ca..edc1421 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,5 +1,8 @@ 2011-05-27 Jason Merrill + PR c++/47687 + * pt.c (dependent_type_p_r): Avoid infinite recursion. + PR c++/48284 * error.c (dump_expr) [COMPONENT_REF]: Use pp_cxx_dot with INDIRECT_REF of REFERENCE_TYPE. diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 71fe0a0..ae3d83d 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -18260,8 +18260,15 @@ dependent_type_p_r (tree type) scope = TYPE_CONTEXT (type); if (scope && TYPE_P (scope)) return dependent_type_p (scope); - else if (scope && TREE_CODE (scope) == FUNCTION_DECL) - return type_dependent_expression_p (scope); + /* Don't use type_dependent_expression_p here, as it can lead + to infinite recursion trying to determine whether a lambda + nested in a lambda is dependent (c++/47687). */ + else if (scope && TREE_CODE (scope) == FUNCTION_DECL + && DECL_LANG_SPECIFIC (scope) + && DECL_TEMPLATE_INFO (scope) + && (any_dependent_template_arguments_p + (INNERMOST_TEMPLATE_ARGS (DECL_TI_ARGS (scope))))) + return true; /* Other types are non-dependent. */ return false; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index d034def..841a202 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,7 @@ 2011-05-27 Jason Merrill + * g++.dg/cpp0x/lambda/lambda-nested4.C: New. + * g++.dg/cpp0x/error6.C: New. * g++.dg/cpp0x/error5.C: New. diff --git a/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-nested4.C b/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-nested4.C new file mode 100644 index 0000000..a5bd1a2 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-nested4.C @@ -0,0 +1,9 @@ +// PR c++/47687 +// { dg-options -std=c++0x } + +template struct A { }; + +auto inl = []{ return []{}; }(); +typedef decltype(inl) inlt; + +A a; -- 2.7.4