<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 24, 2014 at 6:05 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.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">On 02/24/2014 08:53 AM, Alexey Samsonov wrote:<br>
<div class="">> On Fri, Feb 21, 2014 at 9:03 PM, Brad King wrote:<br>
>     Just "build" should be sufficient because the generated build system<br>
>     in compiler-rt knows how to re-run CMake if any of the inputs have<br>
>     changed.<br>
><br>
</div>> Suppose LLVMConfig.cmake has changed.<br>
<br>
Every file CMake processes as part of configuring the project ends up in<br>
the build-time dependencies to re-run CMake.<br></blockquote><div><br></div><div>Alright, I've switched this to</div><div><div>  ExternalProject_Add_Step(compiler-rt force-rebuild</div><div>    DEPENDERS build</div><div>
    ALWAYS 1</div><div>    )</div></div><div><br></div><div>However, looks like this works with Unix Makefiles generators, but doesn't work for Ninja.</div><div>When I run "ninja compiler-rt" twice in the build tree, I see the following message on the second run:</div>
<div>  ninja: no work to do.</div><div>I've verified that compiler-rt is not rebuilt even if I change its sources. Is it a CMake bug?</div><div><br></div><div> </div><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 class=""><br>
> Could you take a look if <a href="http://llvm-reviews.chandlerc.com/D2868" target="_blank">http://llvm-reviews.chandlerc.com/D2868</a> looks reasonable?<br>
<br>
</div>It looks pretty good to me.  I added some inline comments.<br>
<span class=""><font color="#888888"><br>
-Brad<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>