[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 30 16:23:26 PST 2020
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
More information about the cfe-commits
mailing list