jdoerfert added a comment.

In D77683#1970826 <https://reviews.llvm.org/D77683#1970826>, @mehdi_amini wrote:

> I am still not sure what "if someone has asked for extra review of a specific 
> area" refers to?


As said earlier

>>   If I understand this correctly, this is meant to cover situations where 
>> reviewers are active in an area and indicated an interest in reviewing 
>> basically everything.
> 
> Pretty much, yes.

this should mean that, if requested, all non-trivial patches should go through 
review. The current wording is very lenient, especially wrt. code owners.
While most people I talked to don't see owners as special per se (but just 
assume they have more responsibility), this paragraph says they have special 
rights.
Given the murky ways owners are "selected", I think we should have a well 
defined way to limit these rights without revoking the status.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77683



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

Reply via email to