[PATCH] D108123: [MemoryBuiltins] Mark user defined delete as nobuiltin

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

I think this is a clang problem.
What if we are statically linking to the libc++ with LTO?
Surely we'll have said definition avaliable, even though it is not `nobuiltin`?
What if it is overloaded in another TU, or via `LD_PRELOAD`? We then won't think it's `nobuiltin`?


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

https://reviews.llvm.org/D108123