[llvm] [GitHub] Add main LLVM Dialect path to MLIR LLVM PR subscribers (PR #65513)

Markus Böck via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 6 23:43:21 PDT 2023


https://github.com/zero9178 updated https://github.com/llvm/llvm-project/pull/65513:

>From c855e09122e30c494068d83cdba058013ac04828 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Markus=20B=C3=B6ck?= <markus.boeck02 at gmail.com>
Date: Wed, 6 Sep 2023 20:40:37 +0200
Subject: [PATCH 1/2] [GitHub] Add main LLVM Dialect path to MLIR LLVM PR
 subscribers

The LLVM Dialect in MLIR, which the `mlir-llvm` team is supposed to provide notifications for, is 98% not nested in a directory called LLVM but rather LLVMIR. The former only contains some tests.

This should make PRs such as https://github.com/llvm/llvm-project/pull/65508 add the team as codeowner.
---
 .github/CODEOWNERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
index 235e22005b09408..ef64263e1821a7b 100644
--- a/.github/CODEOWNERS
+++ b/.github/CODEOWNERS
@@ -120,6 +120,7 @@
 
 # mlir-llvm
 /mlir/**/LLVM/ @llvm/pr-subscribers-mlir-llvm
+/mlir/**/LLVMIR/ @llvm/pr-subscribers-mlir-llvm
 
 # mlir-linalg
 /mlir/**/*Linalg @llvm/pr-subscribers-mlir-linalg

>From 1b0c83ed2d8b21dba537c9ef06da70fd4b05b00a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Markus=20B=C3=B6ck?= <markus.bock+llvm at nextsilicon.com>
Date: Thu, 7 Sep 2023 08:43:04 +0200
Subject: [PATCH 2/2] Use `LLVM*` patten instead of adding an entry with
 `LLVMIR`

---
 .github/CODEOWNERS | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
index ef64263e1821a7b..1c5e83c034f0a97 100644
--- a/.github/CODEOWNERS
+++ b/.github/CODEOWNERS
@@ -119,8 +119,7 @@
 /mlir/**/Index/ @llvm/pr-subscribers-mlir-index
 
 # mlir-llvm
-/mlir/**/LLVM/ @llvm/pr-subscribers-mlir-llvm
-/mlir/**/LLVMIR/ @llvm/pr-subscribers-mlir-llvm
+/mlir/**/LLVM* @llvm/pr-subscribers-mlir-llvm
 
 # mlir-linalg
 /mlir/**/*Linalg @llvm/pr-subscribers-mlir-linalg



More information about the llvm-commits mailing list