Clang和二进制折叠表达式 - 空参数包的诅咒

时间:2015-08-12 18:18:38

标签: c++ clang compiler-bug c++17 fold-expression

特别是Clang 3.6.0,目前由Coliru主持。

所有这些片段都来自:

int main() {
    foo();
    std::cout << "\n----\n";
    foo(1, 2, 3);
}

以下代码:

template <class... Args>
void foo(Args... args) {
    std::cout << ... << args;
}

触发以下编译错误:

main.cpp:7:17: error: expected ';' after expression
    std::cout << ... << args;
                ^
                ;
main.cpp:7:15: error: expected expression
    std::cout << ... << args;
              ^

所以我尝试在括号中加上括号:

(std::cout << ... << args);

它有效,但会触发警告:

main.cpp:7:6: warning: expression result unused [-Wunused-value]
    (std::cout << ... << args);
     ^~~~~~~~~
main.cpp:11:5: note: in instantiation of function template specialization 'foo<>' requested here
    foo();
    ^

所以我尝试将函数样式转换为void放弃表达式的值:

void(std::cout << ... << args);

但是:

main.cpp:7:20: error: expected ')'
    void(std::cout << ... << args);
                   ^
main.cpp:7:9: note: to match this '('
    void(std::cout << ... << args);
        ^

我也尝试了static_cast,但结果相同。

所以我尝试用C-cast代替:

(void)(std::cout << ... << args);

但是:

main.cpp:6:18: warning: unused parameter 'args' [-Wunused-parameter]
void foo(Args... args) {
                 ^

...我的输出只有----foo(1, 2, 3);不再输出了!

Clang是否受到来自未来标准的邪恶力量的诅咒,它是否有虫子,或者现在坐在我的椅子上的问题是什么?

3 个答案:

答案 0 :(得分:11)

使用功能表示法强制转换为void时,需要一组额外的括号,否则括号将被视为强制转换表达式的一部分,而不是折叠表达式。 fold expression syntax本身需要一组括号。

以下所有工作均未发出任何警告:

void((std::cout << ... << args));
(void)((std::cout << ... << args));

或者只是调用一些ostream成员函数来避免未使用的结果警告

(std::cout << ... << args).flush();

作为T.C.在下面的评论中提到,(void)(std::cout << ... << args);的行为似乎是一个铿锵的错误。演员表示法的语法在 5.4 [expr.cast]

中指定
  

铸表达
  一元表达式
  (type-id)cast-expression

由于括号不是铸造表达式的一部分,因此该用法不应产生警告,更重要的是,它应该导致打印参数。

答案 1 :(得分:3)

我决定更好地了解Clang源中的这个bug。这是有问题的代码部分。这种情况发生在它刚刚完成解析(<type>)并且正在解析以下带括号的表达式时:

} else if (isTypeCast) {
  // Parse the expression-list.
  InMessageExpressionRAIIObject InMessage(*this, false);

  ExprVector ArgExprs;
  CommaLocsTy CommaLocs;

  if (!ParseSimpleExpressionList(ArgExprs, CommaLocs)) {
    // FIXME: If we ever support comma expressions as operands to
    // fold-expressions, we'll need to allow multiple ArgExprs here.
    if (ArgExprs.size() == 1 && isFoldOperator(Tok.getKind()) &&
        NextToken().is(tok::ellipsis))
    return ParseFoldExpression(Result, T);

    ExprType = SimpleExpr;
    Result = Actions.ActOnParenListExpr(OpenLoc, Tok.getLocation(),
                                        ArgExprs);
  }
}

// The beginning of ParseFoldExpression(LHS, T):
if (LHS.isInvalid()) {
  T.skipToEnd();
  return true;
}

负责此错误的代码的具体部分如下:

return ParseFoldExpression(Result, T);

事实证明Result永远不会与其初始true值分开。我认为它应设置为ArgExprs.front(),现在保持std::cout

现在你也会注意到FIXME。虽然特别是与这个问题无关,但可能值得与此同时解决。

作为我的第一个Clang修复程序,在提交更改之前我还有几件事要做(作为参考,Clang 4.0目前正在开发中)。无论是我还是其他人,我都会非常高兴能够解决这个问题。至少,我的研究结果暂时记录在某处。

答案 2 :(得分:2)

来自[expr.prim.fold]的折叠表达式为:

  

折叠表达式对二元运算符执行模板参数包(14.5.3)的折叠   折叠表达
  ( cast-expression fold-operator ...)
  (... fold-operator cast-expression
  ( cast-expression fold-operator ... fold-operator cast-expression

请注意,在所有情况下,括号都是语法的一部分。所以你的初始例子在语法上是不正确的,必须是:

template <class... Args>
void foo(Args... args) {
    (std::cout << ... << args);
}

然后在空包的情况下会给你一个警告,因为二进制折叠减少到只有std::cout;要摆脱那个警告,你可以按照通常的方式转换到{{1} - 只是内部括号集是语法的一部分所以你需要两个

void

或者你可以加一个额外的void((std::cout << ... << args)); 之类的东西:

endl

或者返回结果:

(std::cout << ... << args) << std::endl;