[PATCH] D71490: [lit] [windows] Make sure to convert all path separators to backslashes in NT style \\?\... paths
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 13:55:51 PST 2019
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
lgtm
================
Comment at: llvm/utils/lit/lit/util.py:157
+ # NT path format only supports backslashes.
+ path = '\\'.join(path.split('/'))
NTPath = to_unicode(r'\\?\%s' % path)
----------------
I feel like path.replace('/', '\\') would be the more obvious spelling (fewer temporary strings), but they are functionally equivalent.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71490/new/
https://reviews.llvm.org/D71490
More information about the llvm-commits
mailing list