[llvm] Remove CODEOWNERS file and copy missing paths into new-prs-labeler.yml (PR #66145)

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 13 16:05:40 PDT 2023


================
@@ -183,7 +184,8 @@ mlir:index:
   - mlir/**/Index/**
 
 mlir:llvm:
-  - mlir/**/LLVM/**
+  - mlir/**/LLVM*
+  - mlir/**/LLVM*/**
----------------
tstellar wrote:

Yes.  Here is the small js program I was using to check the patterns:
```
const { minimatch } = require('minimatch')

function check_pattern(file, pattern) {
    match = minimatch(file, pattern)
    console.log(file + " : " + pattern + " : " + match)
}
check_pattern('mlir/foo/LLVM.cpp', 'mlir/**/LLVM*')
check_pattern('mlir/foo/LLVM.cpp', 'mlir/**/LLVM*/**')
check_pattern('mlir/foo/LLVM/a.cpp', 'mlir/**/LLVM*')
check_pattern('mlir/foo/LLVM/a.cpp', 'mlir/**/LLVM*/**')
```
```
mlir/foo/LLVM.cpp : mlir/**/LLVM* : true
mlir/foo/LLVM.cpp : mlir/**/LLVM*/** : false
mlir/foo/LLVM/a.cpp : mlir/**/LLVM* : false
mlir/foo/LLVM/a.cpp : mlir/**/LLVM*/** : true
```

https://github.com/llvm/llvm-project/pull/66145


More information about the llvm-commits mailing list