hans added a comment.

If I understand correctly, D68028 <https://reviews.llvm.org/D68028> made it so 
that LLVM doesn't add any builtin calls (e.g. turning for-loops into memcpy), 
but Clang could still turn calls into builtins. Maybe the patch description 
could be expanded to explain this?



================
Comment at: clang/lib/CodeGen/CGExpr.cpp:5219
+    // function.
+    else if (!CGF.CurFn->getAttributes().hasFnAttr(AttributeNoBuiltin))
       return CGCallee::forBuiltin(builtinID, FD);
----------------
What if CurFn has the "wildcard" no-builtins attribute?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124701/new/

https://reviews.llvm.org/D124701

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to