[PATCH] D73543: [clang] Add support for __builtin_memcpy_inline
    Eli Friedman via Phabricator via llvm-commits 
    llvm-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 llvm-commits
mailing list