[PATCH] D80531: [clang-tidy]: Added modernize-replace-disallow-copy-and-assign-macro

Konrad Wilhelm Kleine via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 2 14:16:19 PDT 2020


kwk added a comment.

In D80531#2069383 <https://reviews.llvm.org/D80531#2069383>, @njames93 wrote:

> LGTM with 2 small nits, but I'd still give a few days see if anyone else wants to weight in.


I'm okay with this but I'd like to understand when it's time to wait for others? Only when a patch is approved or from the beginning of patch's life? I mean who looks at patches that are approved unless you filter for the amount of approvers? How many approvers must there be?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80531





More information about the cfe-commits mailing list