<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Dec 28, 2013 at 4:31 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="im"><br>
<div class="gmail_quote">On Sat, Dec 28, 2013 at 6:28 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Since I won't have access to a windows or linux box until <span><span>monday</span></span>, this addresses a FIXME and blocks deadstripping for the cmake build, I'll land this for post-commit review and hope that the bots will help me sort things out. (I tested the non-darwin branches on my os x box by temporarily changing the conditionals, they seem to do what they're supposed to do.)</div>

</blockquote></div><br></div>SGTM. Sorry for not replying sooner, I sent my only comment in post-commit. I'm very confident the more interesting thing is sorting out how to make this work on all the platforms which I see you're doing. Thanks!</div>

</div>
</blockquote></div><br></div><div class="gmail_extra">It looks like everything except windows is working. There were a few good suggestions on IRC, I'll take another look at windows in a few hours.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">If someone else gets around to it before me: cmake/modules/AddLLVM.cmake around line 40 is where the command comes from, and cmd.exe complains about:</div><div class="gmail_extra"><br></div>
<div class="gmail_extra"><div class="gmail_extra">FAILED: cmd.exe /c cd /D C:\bb-win7\cmake-clang-i686-mingw32\build\tools\lto && "c:\Program Files (x86)\CMake 2.8\bin\cmake.exe" -E echo EXPORTS > symbol.def && type C:/bb-win7/cmake-clang-i686-mingw32/llvm-project/llvm/tools/lto/lto.exports >> symbol.def</div>
<div class="gmail_extra">The syntax of the command is incorrect.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Replacing the /s with \s apparently makes things work, so if the bot is still red in a few hours I'll try inserting a FILE(TO_NATIVE_PATH call.</div>
</div><div class="gmail_extra"><br></div><div class="gmail_extra">( <a href="http://bb.pgr.jp/builders/cmake-clang-i686-mingw32">http://bb.pgr.jp/builders/cmake-clang-i686-mingw32</a> )</div></div>