<div class="gmail_quote">On Mon, Jul 30, 2012 at 4:10 PM, Tom Stellard <span dir="ltr"><<a href="mailto:thomas.stellard@amd.com" target="_blank">thomas.stellard@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Jul 30, 2012 at 03:01:40PM -0400, Justin Holewinski wrote:<br>
> The only thing I would add (other than the autotools version), is a<br>
> more descriptive error message for when a staged back-end is added<br>
> to LLVM_TARGETS_TO_BUILD without setting LLVM_ENABLE_STAGING.<br>
><br>
<br>
Here is version 2 of the cmake patch with a more descriptive error<br>
message and also a new patch that adds the --enable-staging-targets option<br>
to the configure script.<br></blockquote><div><br></div><div>Looks good to me, with the caveat that my autoconf-fu is pretty weak.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
-Tom<br>
<br>
<br>
> On 07/30/2012 02:10 PM, Tom Stellard wrote:<br>
> >Hi,<br>
> ><br>
> >The attached patched adds the LLVM_ENABLE_STAGING flag to CMake.  This<br>
> >flag is disabled by default and must be enabled to build staging<br>
> >targets.  There are no staging targets at the moment, so the flag<br>
> >doesn't actually do anything yet.<br>
> ><br>
> >The idea for this patch comes from the LLVM staging area discussion thread[1]<br>
> >on LLVM dev.<br>
> ><br>
> >Please Review.<br>
> ><br>
> >Thanks,<br>
> >Tom<br>
> ><br>
> >[1] <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-July/052146.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-July/052146.html</a><br>
> ><br>
> ><br>
> >_______________________________________________<br>
> >llvm-commits mailing list<br>
> ><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> ><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<span class="HOEnZb"><font color="#888888">><br>
> --<br>
> Thanks,<br>
><br>
> Justin Holewinski<br>
><br>
<br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>