rsmith accepted this revision.
rsmith added inline comments.
This revision is now accepted and ready to land.

================
Comment at: clang/lib/AST/ExprConstant.cpp:1871-1902
       if (!CheckConstantExpression(Info, DiagLoc, EltTy,
                                    Value.getArrayInitializedElt(I)))
         return false;
     }
     if (!Value.hasArrayFiller())
       return true;
     return CheckConstantExpression(Info, DiagLoc, EltTy,
----------------
`Usage`  should be passed into these recursive calls to 
`CheckConstantExpression`, although that's NFC for now, until/unless we start 
allowing class types as template parameters.


https://reviews.llvm.org/D43320



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

Reply via email to