[llvm] [polly] [TableGen] Refactor Intrinsic handling in TableGen (PR #103980)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 16 02:25:11 PDT 2024
jayfoad wrote:
> What's the convention in terms of approvals needed? Do we need atleast 2 approvals or something before merging the change? In any case there is an unresolved thread above, so will wait, but wanted to know.
See https://llvm.org/docs/CodeReview.html#lgtm-how-a-patch-is-accepted but to summarize, one approval is enough so long as there are no outstanding concerns from other reviewers.
https://github.com/llvm/llvm-project/pull/103980
More information about the llvm-commits
mailing list