[PATCH] D30514: [libc++abi] Update new/delete definitions to match libc++

Jonathan Roelofs via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 2 08:36:05 PST 2017


jroelofs added a comment.

In https://reviews.llvm.org/D30514#690318, @mehdi_amini wrote:

> LGTM.
>
> (It seems that having libc++ and libc++abi in the same repo would help sharing code like this)


I think it would be a step backwards in terms of opening things up for layering violations. We already have one unavoidable one that comes from they way typeinfo matching works in ARM EHABI... I'd be afraid of it getting worse.


https://reviews.llvm.org/D30514





More information about the cfe-commits mailing list