martong added a comment.

In D126215#3565572 <https://reviews.llvm.org/D126215#3565572>, @steakhal wrote:

> I believe, the silence from the code owner means that he agrees with this 
> change.
> I'll land it tomorrow.

I believe there is a misconception about the role of a code owner, we cannot 
expect Artem to review all patches. According to the LLVM Developer Poicy

> The sole responsibility of a code owner is to ensure that a commit to their 
> area of the code is appropriately reviewed, either by themself or by 
> **someone else**.

Many other analyzer pathces in the past had been reviewed by me and you 
@steakhal and we didn't receive any complaints after they landed. So, I think 
it is okay to land this one as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126215

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to