From c855e09122e30c494068d83cdba058013ac04828 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Markus=20B=C3=B6ck?= 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 235e22005b094..ef64263e1821a 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?= 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 ef64263e1821a..1c5e83c034f0a 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