[libcxx-commits] [PATCH] D57262: [libunwind] Drop the dependency on <algorithm>, add placement new inline

[Petr Hosek via Phabricator via libcxx-commits]

phosek marked an inline comment as done.
phosek added inline comments.


================
Comment at: libunwind/src/libunwind.cpp:30
 
+inline void* operator new(size_t, void* __p) { return __p; }
+
----------------
ldionne wrote:
> Why do we need to define this? I thought placement new was handled by Clang directly, without requiring a declaration?
That's what I assumed but that doesn't seem to be the case, without this definition the compilation fails because Clang cannot find the operator `new`.


Repository:
  rUNW libunwind

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

https://reviews.llvm.org/D57262