[PATCH] D61121: [Windows] Separate elements in -print-search-dirs with semicolons
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 25 10:34:10 PDT 2019
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
In D61121#1479031 <https://reviews.llvm.org/D61121#1479031>, @smeenai wrote:
> I feel something like `llvm::sys::path::PathListSeparator` would be nicer, since AFAIK any path list should be separated with semicolons on Windows and colons everywhere else.
I think the dominant use case for `EnvPathSeparator` is separating a list of paths in environment variables. We have other use cases for it in lists of paths that aren't environment variables (-print-search-dirs), but renaming it just seems like a minor improvement, which seems like it should be separate from this change.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61121/new/
https://reviews.llvm.org/D61121
More information about the cfe-commits
mailing list