rsmith added a subscriber: rsmith.

================
Comment at: lib/Lex/PPMacroExpansion.cpp:1438-1440
@@ -1432,3 +1437,5 @@
 
-  // Get the module name.
-  PP.LexNonComment(Tok);
+  // It is possible that the macro invocation has a multiple-nested argument,
+  // for example: __has_feature((((X)))).  This should be counted as valid and
+  // parsed correctly.
+  SmallVector<SourceLocation, 2> ParenStack;
----------------
Why should this be treated as valid? That's not part of the specification for 
these features.


http://reviews.llvm.org/D17149



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

Reply via email to