[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