commit 83d4834597a779ccafbce6cd91257f4c712adf5f
author Richard Smith <richard-llvm@metafoo.co.uk> Fri Nov 15 02:29:26 2013 +0000
committer Richard Smith <richard-llvm@metafoo.co.uk> Fri Nov 15 02:29:26 2013 +0000
tree 2a39c0d7127d072f9c84903876b941c59a3896f5
parent 09e59155ef772afe59a47aa2a309d54003af814c

Don't reject dependent range-based for loops in constexpr functions. The loop
variable isn't really uninitialized, it's just not initialized yet.

llvm-svn: 194767

clang/lib/Sema/SemaDeclCXX.cpp

diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp
index 660641e..1eec231 100644
--- a/clang/lib/Sema/SemaDeclCXX.cpp
+++ b/clang/lib/Sema/SemaDeclCXX.cpp
@@ -880,7 +880,7 @@
               diag::err_constexpr_local_var_non_literal_type,
               isa<CXXConstructorDecl>(Dcl)))
           return false;
-        if (!VD->hasInit()) {
+        if (!VD->hasInit() && !VD->isCXXForRangeDecl()) {
           SemaRef.Diag(VD->getLocation(),
                        diag::err_constexpr_local_var_no_init)
             << isa<CXXConstructorDecl>(Dcl);