================ @@ -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