[libcxx-commits] [PATCH] D75795: [libc++abi] Change __cxa_finalize return type to void

Roman Lebedev via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Mar 10 15:22:04 PDT 2020


lebedev.ri added a comment.

In D75795#1915802 <https://reviews.llvm.org/D75795#1915802>, @ldionne wrote:

> Folks, it's not okay to commit changes to libc++ without having a libc++ or libc++abi code owner review them. If you wait for us to review the change and a review is not coming, you can ping the patch. But you can't commit it just because 2 business days have passed and we haven't said anything.
>
> @MaskRay This isn't the first time this happens. The LLVM community is very loose by giving global commit access to everybody, but we're expecting people to act in good faith and not bypass the usual review process.


Was phabricator-friendly approach considered?
Why not have a herald rule to auto-put someone from libc++ as blocking reviewer on new libc++ patches instead of such trigger-happy reverts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75795





More information about the libcxx-commits mailing list