I actually think that should live in lldb.  All of llvm's path handling code is stateless which makes the most sense from an api perspective.  Lldb is just "special" in that regard <br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 13, 2017 at 3:29 AM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Lldb has a copy of a part of this code with the same type of `#ifdef` -> `if` changes applied, so it would be great it we could get rid of that.<br class="gmail_msg">
<br class="gmail_msg">
What concerns me about this from a usability point of view is that the path style is not stored explicitly anywhere -- it just floats around implicitly. For working with remote paths, you're going to need some sort of an object which encapsulates this information. I suppose this could live in lldb's FileSpec as it did until now, but I'm wondering whether we shouldn't add something like that here, if we're going to start advertising foreign path support. I am asking that now because that would affect the signature of all of these functions, and it's probably better to avoid changing them twice.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: llvm/unittests/Support/Path.cpp:62<br class="gmail_msg">
 #else<br class="gmail_msg">
-  EXPECT_FALSE(path::is_separator('\\'));<br class="gmail_msg">
+  EXPECT_FALSE(path::is_separator('\\', ));<br class="gmail_msg">
 #endif<br class="gmail_msg">
----------------<br class="gmail_msg">
stray comma<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D30858" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D30858</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>