cor3ntin added inline comments.
================ Comment at: clang/lib/AST/Interp/Interp.h:1902 + std::vector<int64_t> ArrayIndices; + for (unsigned I = 0; I != E->getNumExpressions(); ++I) + ArrayIndices.emplace_back(S.Stk.pop<int64_t>()); ---------------- can you use size_t here? It's already a bug nest but it can't hurt to use better types, even if `getNumExpressions` returns unsigned https://godbolt.org/z/vhYxose4o CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156400/new/ https://reviews.llvm.org/D156400 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
