================
@@ -231,6 +233,10 @@ bool GlobPattern::SubGlobPattern::match(StringRef Str) 
const {
         ++S;
         continue;
       }
+    } else if (IsSlashAgnostic && *P == '/' && (*S == '/' || *S == '\\')) {
----------------
jyknight wrote:

`*S == '/'` is redundant here and can be removed; it'd be handled in the next 
clause anyhow.

For completeness, it may make sense on windows to also match `\\` against `/`. 
Then we can document as "now treats forward and backward slashes as equivalent 
when matching paths"

https://github.com/llvm/llvm-project/pull/149886
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to