[PATCH] D126215: [analyzer] Deprecate `-analyzer-store region` flag

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 8 02:00:13 PDT 2022


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



More information about the cfe-commits mailing list