<div dir="ltr"><br><div class="gmail_quote">On Wed Feb 04 2015 at 10:39:58 AM Robert Flack <<a href="mailto:flackr@gmail.com">flackr@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D7348#118521" target="_blank">http://reviews.llvm.org/D7348#<u></u>118521</a>, @zturner wrote:<br>
<br>
> Hi Robert, by any chance did you actually try compiling this on Windows?<br>
>  The reason I ask is that it just occurred to me that this will now require<br>
>  linking in rpcrt4.lib, which I don't think happens by default.  So this<br>
>  will cause linker errors.  If you tested it and it compiled, then no<br>
>  problem.  But if you didn't test it, then I think it's better to just go<br>
>  back to the way it was before, and I will do the UuidToString stuff later.<br>
<br>
<br>
I did, but it didn't get as far as the linking phase due to an unrelated error:<br>
<br>
D:\src\ll\llvm\tools\lldb\<u></u>source\Target\<u></u>ProcessLaunchInfo.cpp(347) : error C2065: 'O_CLOEXEC' : undeclared identifier<br>
[125/270] Building CXX object tools\ll...iles\lldbTarget.<u></u>dir\ThreadPlan.cpp.obj<br>
ninja: build stopped: subcommand failed.<br>
<br>
So I'm not sure if there will be linker errors. I can go back to the way it was before on this.<br></blockquote>Sounds good. I'll look into fixing this error.  I'm syncing now to see what the status is of compiling at ToT.  I like your idea of not storing the name, and just returning it as an output parameter.  Not everyone will care about the name, so for them this doesn't impose a penalty on them.  Feel free to change it to that way instead.  The usual paradigm for this is to have the method take an llvm::SmallVector<char> & as an argument.  clear() it then just write the value into it.  Callers will allocate a SmallString<N> and pass it in by reference.</div></div>