rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land.
Thanks! ================ Comment at: lib/Parse/ParseExpr.cpp:273 static bool isFoldOperator(tok::TokenKind Kind) { - return isFoldOperator(getBinOpPrecedence(Kind, false, true)); + return isFoldOperator(getBinOpPrecedence(Kind, true, true)); } ---------------- Do we ever need to pass in the `Parser`'s `GreaterThanIsOperator` value here? (From a quick check, it looks like we never call `isFoldOperator` in any case where `>` might end a template argument list instead, but it's not completely obvious, because we do support parsing unparenthesized fold operators for better error handling in some cases.) Seems like it would be safer to pass the correct value in here, even if it happens to not matter right now. https://reviews.llvm.org/D36101 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits