[PATCH] D30514: [libc++abi] Update new/delete definitions to match libc++
mehdi_amini added a comment. In https://reviews.llvm.org/D30514#690702, @jroelofs wrote: > 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. I don't see why: this is about discipline and bots. Enforcing layering using separate repository sounds awful to me. https://reviews.llvm.org/D30514 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D30514: [libc++abi] Update new/delete definitions to match libc++
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 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D30514: [libc++abi] Update new/delete definitions to match libc++
mehdi_amini accepted this revision. mehdi_amini added a comment. This revision is now accepted and ready to land. LGTM. (It seems that having libc++ and libc++abi in the same repo would help sharing code like this) https://reviews.llvm.org/D30514 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits