[PATCH] D43320: Allow dllimport non-type template arguments in C++17

[Richard Smith - zygoloid via Phabricator via cfe-commits]

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