[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