<div dir="ltr">Thanks.  Also my patch up there was wrong anyway, so it all worked out in the end :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 19, 2014 at 8:38 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> writes:<br>

> Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> writes:<br>
>> This generates some crashes when running Windows tests.  In particular, this<br>
>> asserts is being hit:<br>
>><br>
>>   assert(sys::path::is_absolute(VirtualPath) && "virtual path not absolute");<br>
>><br>
>> inside the function void YAMLVFSWriter::addFileMapping(StringRef VirtualPath,<br>
>> StringRef RealPath)<br>
>><br>
>> Inspecting VirtualPath in a debugger shows that the value is:<br>
>><br>
>> \\src\\llvm\\tools\\clang\\test\\Modules\\Inputs\\Module.framework\\Frameworks<br>
>> \\SubFramework.framework\\Headers\\SubFramework.h<br>
>><br>
>> The path qualified with drive letter on my machine that this resolves to is<br>
>> the aforementioned path prefixed with a D:<br>
>><br>
>> So this should refer to a root path, but either way I'll leave it to you to<br>
>> decide whether the fix is to append the drive letter, or maybe the bug is in<br>
>> the is_absolute() function.<br>
><br>
> The copyToRoot function is wrong here - It takes a shortcut in working<br>
> out an absolute path that isn't valid on windows. I'll come up with a<br>
> fix in a couple of hours.<br>
<br>
</div></div>Should be fixed in r211338.<br>
</blockquote></div><br></div>