rsmith added a comment.

Thanks, this essentially looks good to me. I can't think of any other cases 
where C++ allows `auto` that you've not covered.


================
Comment at: lib/Sema/SemaExprCXX.cpp:1172-1173
@@ -1171,1 +1171,4 @@
 
+  if (D.getDeclSpec().getTypeSpecType() == DeclSpec::TST_auto_type)
+      return ExprError(Diag(StartLoc, diag::err_new_auto_type));
+
----------------
How about instead handling this...

================
Comment at: lib/Sema/SemaType.cpp:2687
@@ -2682,3 +2686,3 @@
     case Declarator::ConditionContext:
     case Declarator::CXXNewContext:
       break;
----------------
... here.


http://reviews.llvm.org/D12686



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to