From efd115c1c6e42f689659802e4eab60c455a53579 Mon Sep 17 00:00:00 2001 From: Jason Merrill Date: Wed, 11 May 2011 17:29:56 -0400 Subject: [PATCH] * decl.c (grokdeclarator): Only set DECL_DECLARED_CONSTEXPR_P once. From-SVN: r173680 --- gcc/cp/ChangeLog | 2 ++ gcc/cp/decl.c | 12 +++++++----- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 746b69de1d6..d6241cd4a2c 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,5 +1,7 @@ 2011-05-11 Jason Merrill + * decl.c (grokdeclarator): Only set DECL_DECLARED_CONSTEXPR_P once. + * pt.c (build_non_dependent_expr): Don't check null_ptr_cst_p, do call maybe_constant_value in C++0x mode. * semantics.c (cxx_eval_constant_expression): Handle TEMPLATE_DECL. diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index eff23608b6a..ad816f16977 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -9933,7 +9933,6 @@ grokdeclarator (const cp_declarator *declarator, return error_mark_node; } - DECL_DECLARED_CONSTEXPR_P (decl) = constexpr_p; decl = do_friend (ctype, unqualified_id, decl, *attrlist, flags, funcdef_flag); @@ -10183,8 +10182,11 @@ grokdeclarator (const cp_declarator *declarator, } } else if (constexpr_p && DECL_EXTERNAL (decl)) - error ("declaration of constexpr variable %qD is not a definition", - decl); + { + error ("declaration of constexpr variable %qD is not a definition", + decl); + constexpr_p = false; + } } if (storage_class == sc_extern && initialized && !funcdef_flag) @@ -10213,8 +10215,8 @@ grokdeclarator (const cp_declarator *declarator, else if (storage_class == sc_static) DECL_THIS_STATIC (decl) = 1; - /* Don't forget constexprness. */ - if (constexpr_p) + /* Set constexpr flag on vars (functions got it in grokfndecl). */ + if (constexpr_p && TREE_CODE (decl) == VAR_DECL) DECL_DECLARED_CONSTEXPR_P (decl) = true; /* Record constancy and volatility on the DECL itself . There's -- 2.30.2