<div class="gmail_quote">On Thu, Feb 16, 2012 at 7:07 PM, Sebastian Pop <span dir="ltr"><<a href="mailto:spop@codeaurora.org">spop@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Daniel,<br>
<div class="im"><br>
On Thu, Nov 3, 2011 at 5:46 PM, Daniel Dunbar <<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>> wrote:<br>
</div><div><div class="h5">> Author: ddunbar<br>
> Date: Thu Nov  3 17:46:21 2011<br>
> New Revision: 143662<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=143662&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=143662&view=rev</a><br>
> Log:<br>
> build/Make: Integrate llvm-build into Makefiles.<br>
>  - Basically, we coordinate with llvm-build to create a Makefile fragment we can<br>
>   easily use. For now, nothing is wired in except the support to automatically<br>
>   regenerate this file when necessary.<br>
><br>
> Modified:<br>
>    llvm/trunk/Makefile.rules<br>
><br>
> Modified: llvm/trunk/Makefile.rules<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/Makefile.rules?rev=143662&r1=143661&r2=143662&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/Makefile.rules?rev=143662&r1=143661&r2=143662&view=diff</a><br>

> ==============================================================================<br>
> --- llvm/trunk/Makefile.rules (original)<br>
> +++ llvm/trunk/Makefile.rules Thu Nov  3 17:46:21 2011<br>
> @@ -57,6 +57,54 @@<br>
><br>
>  $(UserTargets)::<br>
><br>
> +#------------------------------------------------------------------------<br>
> +# LLVMBuild Integration<br>
> +#------------------------------------------------------------------------<br>
> +#<br>
> +# We use llvm-build to generate all the data required by the Makefile based<br>
> +# build system in one swoop:<br>
> +#<br>
> +#  - We generate a file (a Makefile fragment) in the object root which contains<br>
> +#    all the definitions that are required by Makefiles across the entire<br>
> +#    project.<br>
> +#<br>
> +#  - We generate the library table used by llvm-config.<br>
> +#<br>
> +#  - We generate the dependencies for the Makefile fragment, so that we will<br>
> +#    automatically reconfigure outselves.<br>
> +<br>
> +# The path to the llvm-build tool itself.<br>
> +LLVMBuildTool  := $(PROJ_SRC_ROOT)/utils/llvm-build/llvm-build<br>
<br>
</div></div>This line most likely should be:<br>
<br>
LLVMBuildTool  := $(LLVM_SRC_ROOT)/utils/llvm-build/llvm-build<br>
<br>
otherwise projects like Polly would fail to find llvm-build in their<br>
utils subdir: i.e., llvm/tools/polly/utils/llvm-build/llvm-build<br>
<br>
With the attached patch polly configures and builds again properly.<br>
Ok to commit?<br></blockquote><div><br></div><div>My only question is do we need to retain a dependency on the project-specific Makefile.rules in addition to the top-level Makefile.rules?</div></div>