[PATCH] D124701: [clang] Honor __attribute__((no_builtin("foo"))) on functions

[Hans Wennborg via Phabricator via cfe-commits]

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