[PATCH] D12686: Add support for GCC's '__auto_type' extension.

Richard Smith via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 9 13:10:35 PST 2015


rsmith added inline comments.

================
Comment at: lib/Sema/SemaType.cpp:2621
@@ -2620,3 +2620,3 @@
   // C++14 In generic lambdas allow 'auto' in their parameters.
-  if (ContainsPlaceholderType &&
+  if (D.getDeclSpec().containsPlaceholderType() &&
       (!SemaRef.getLangOpts().CPlusPlus11 || !D.isFunctionDeclarator())) {
----------------
rsmith wrote:
> Does this still do the right thing for the `IK_ConversionFunctionId` case? It doesn't look like it will: in that case, the `auto` is in the //declarator-id//, not in the //decl-specifier-seq//.
Disregard this comment; I tried to delete it but it looks like phabricator submitted it anyway.


http://reviews.llvm.org/D12686





More information about the cfe-commits mailing list