[libc-commits] [libc] [libc] Change the GPU to use builtin memory functions (PR #68003)

Guillaume Chatelet via libc-commits libc-commits at lists.llvm.org
Tue Oct 3 07:47:43 PDT 2023


gchatelet wrote:

> I actually tried that, but it seemed to cause some bizarre error on AMDGPU. Was planning on leaving it unchanged until I filed a bug and figured out why it crashed.

Ok so maybe remove `inline_memcmp_builtin` in the meantime.

> `-ffreestanding` implies `-fno-builtins` but `-fno-builtins` does not prevent us from using builitin functions. The purpose of `-fno-builtins` is to prevent compiler transformations, for example `-fno-builtin` prevents `memcpy` from becoming `llvm.memcpy`, builtins used by the user are just fine.

Ha indeed!

https://github.com/llvm/llvm-project/pull/68003


More information about the libc-commits mailing list