Ahh that's unfortunate.  I guess the thing to do is make SetFile and SetDirectory.  At least by changing the return type we can get the compiler to tell us everywhere this is happening.  Did you run the test suite on Windows to make sure this doesnt break anything?<br><div class="gmail_quote"><div dir="ltr">On Wed, Apr 13, 2016 at 8:36 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 a comment.<br>
<br>
It's a bit more complicated than it seems. GetDirectory() returns a reference into the FileSpec, and a lot of code modifies it through it. Changing that would require fixups in all of the users, which I'd want to do separately.<br>
<br>
But I do agree that this is the best way forward.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D19060" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19060</a><br>
<br>
<br>
<br>
</blockquote></div>