[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