efriedma added inline comments.
================ Comment at: clang/lib/Sema/SemaChecking.cpp:1653 + llvm::APSInt I(64); + if (!TheCall->getArg(2)->isIntegerConstantExpr(I, Context)) + Diag(TheCall->getBeginLoc(), diag::err_expr_not_ice) ---------------- SemaBuiltinConstantArg. Or actually, you should probably use "I" in Builtins.def. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D73543/new/ https://reviews.llvm.org/D73543 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits