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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

I was really talking about statically linking to libc++ which is also statically linked, and compiling everything with LTO,
then you should have the definition available. I don't know if that is a supported configuration.

If this is a clang bug, then presumably it should be fixed in clang, and not workarounded in middle-end,
penalizing everyone else who doesn't wish to change semantics in such a way,
Wasn't there some command-line argument to specify that a certain builtin isn't a builtin?


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

https://reviews.llvm.org/D108123