<div dir="ltr">Thanks. It looks like UnixSignals.cpp is compiled on Windows, therefore this should too.  For whatever reason, the list of files to compile for the API are maintained in a different CMake location on Windows than on non-Windows.  So when adding a file to API, you need to add it in two places.  Don't revert, I'll just check in a fix that compiles on Windows.  Already tested it locally, just wanted to confirm your intent first.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 24, 2014 at 2:25 PM, Russell Harmon <span dir="ltr"><<a href="mailto:eatnumber1@google.com" target="_blank">eatnumber1@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 class="">It should be just as platform dependant as the UnixSignals object it relies<br>
on. I didn't originally write it with windows in mind.<br>
<br>
What's standard procedure here? Rollback? Or fix forward?<br>
<br>
Without looking at the code, how does Windows deal with the UnixSignals<br>
object? Is it just not built?<br>
<br>
</div><a href="http://reviews.llvm.org/D4221" target="_blank">http://reviews.llvm.org/D4221</a><br>
<br>
<br>
</blockquote></div><br></div>