[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