<div dir="ltr">Bump,<div><br></div><div>Normally I'd either wait longer for a review (I heard things are hectic right now) or just submit this patch (doesn't seem very risky), but the Windows build is currently broken without it, and it modifies the public API which I've never done before, so I want to make sure this is ok.  I'm pretty sure it's fine, since all I've done is added a new method to the public api, but still better safe than sorry.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 1:46 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sorry, here's a more easily digestable patch.</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jul 31, 2014 at 1:45 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">mkstemp doesn't exist on Windows.  LLVM already has a support function which is the same in spirit as mkstemp.  This patch adds a method to the host layer to create a temporary file, and the implementation of this method on the private side delegates to the corresponding LLVM support function.<div>


<br></div><div>This fixes the build on Windows and should have identical semantics on other platforms, but please verify.</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>