Yea, because of the fact that we must support both syntaxes on both platforms, LLVM library is out.<br><br>The whole motivation for introducing the path syntax is so that windows paths behave as if on Windows even if on linux<br><div class="gmail_quote"><div dir="ltr">On Wed, Mar 16, 2016 at 6:19 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added inline comments.<br>
<br>
================<br>
Comment at: lldb/trunk/unittests/Host/FileSpecTest.cpp:25<br>
@@ +24,3 @@<br>
+    EXPECT_STREQ("F:\\bar", fs_windows.GetCString());<br>
+    EXPECT_STREQ("F:", fs_windows.GetDirectory().GetCString());<br>
+    EXPECT_STREQ("bar", fs_windows.GetFilename().GetCString());<br>
----------------<br>
So this returns `F:` on linux and `F:\` on Windows. This happens because `llvm::sys::path::parent_path` does not recognize `F:` as a "root directory" on linux, and therefore treats it differently. I don't know which behavior is more "correct" (probably the windows one), but I think that this should be consistent, regardless of the platform the test is run on (my original motivation for writing this was the fact that i was getting wonky paths while attempting to write other unit tests). Unfortunately, I think that means getting rid of llvm's path processing library...<br>
<br>
What do you make of that?<br>
<br>
(I am going on holiday, so I cannot to anything about this now. if you want to have a clean test run in the mean time, I am fine commenting these checks out or something...)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D18044" rel="noreferrer" target="_blank">http://reviews.llvm.org/D18044</a><br>
<br>
<br>
<br>
</blockquote></div>