[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