FWIW, the cmake here LGTM. I'll leave the autoconf etc to others...<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 8, 2012 at 4:39 PM, Oliveira, Victor <span dir="ltr"><<a href="mailto:Victor.Oliveira@amd.com" target="_blank">Victor.Oliveira@amd.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sure, it makes sense.<br>
<br>
Here is another version of the patch that adds a LLVM_EXPERIMENTAL_TARGETS_TO_BUILD variable to CMake.<br>
________________________________________<br>
From: Justin Holewinski [<a href="mailto:justin.holewinski@gmail.com">justin.holewinski@gmail.com</a>]<br>
Sent: Wednesday, August 08, 2012 1:42 PM<br>
To: Oliveira, Victor<br>
Cc: Eric Christopher; Villmow, Micah; <a href="mailto:David@dcs-maillist.cs.uiuc.edu">David@dcs-maillist.cs.uiuc.edu</a>; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>; Stellard, Thomas; Röthlisberger<br>

<div class="HOEnZb"><div class="h5">Subject: Re: [llvm-commits] PATCH: Add LLVM_ENABLE_STAGING option to CMake and configure<br>
<br>
So we no longer want an ENABLE_EXPERIMENTAL flag in the CMake system, just add the target to LLVM_TARGETS_TO_BUILD?  It seems the consensus for auto tools is to maintain two separate lists and combine them in the end.  Maybe we should do the same in CMake?  We could have LLVM_TARGETS_TO_BUILD and LLVM_EXPERIMENTAL_TARGETS_TO_BUILD, and merge them ins to LLVM_TARGETS_TO_BUILD at configure-time.  I'm all for maintaining consistency between the auto tools and CMake build systems.<br>

<br>
On Aug 7, 2012, at 1:33 PM, "Oliveira, Victor" <<a href="mailto:Victor.Oliveira@amd.com">Victor.Oliveira@amd.com</a>> wrote:<br>
<br>
> Hi!<br>
><br>
> This patch just adds a --enable-experimental-targets=x,y,z option in configure. In CMake, you have to explicitly add the backend names in TARGETS_TO_BUILD (the default option -- all -- just builds non-experimental backends).<br>

><br>
> What do you think?<br>
> Victor<br>
> ________________________________________<br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [<a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a>] on behalf of Eric Christopher [<a href="mailto:echristo@apple.com">echristo@apple.com</a>]<br>

> Sent: Friday, August 03, 2012 10:30 AM<br>
> To: Villmow, Micah<br>
> Cc: <a href="mailto:David@dcs-maillist.cs.uiuc.edu">David@dcs-maillist.cs.uiuc.edu</a>; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>; Stellard, Thomas; Röthlisberger<br>
> Subject: Re: [llvm-commits] PATCH: Add LLVM_ENABLE_STAGING option to CMake and configure<br>
><br>
> The docs seem to be duplicated twice and I might prefer just a simple --enable-experimental-targets=x,y,z just like --enable-targets=x,y,z and just add those to TARGETS_TO_BUILD.<br>
><br>
> -eric<br>
><br>
> On Aug 3, 2012, at 10:12 AM, "Villmow, Micah" <<a href="mailto:Micah.Villmow@amd.com">Micah.Villmow@amd.com</a>> wrote:<br>
><br>
>> Ping! Any autotools users that can comment on this patch?<br>
>><br>
>><br>
>> Thanks,<br>
>> Micah<br>
>><br>
>>> -----Original Message-----<br>
>>> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-">llvm-commits-</a><br>
>>> <a href="mailto:bounces@cs.uiuc.edu">bounces@cs.uiuc.edu</a>] On Behalf Of Tom Stellard<br>
>>> Sent: Thursday, August 02, 2012 6:30 AM<br>
>>> To: Justin Holewinski<br>
>>> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>; Röthlisberger; David@dcs-<br>
>>> <a href="http://maillist.cs.uiuc.edu" target="_blank">maillist.cs.uiuc.edu</a><br>
>>> Subject: Re: [llvm-commits] PATCH: Add LLVM_ENABLE_STAGING option to<br>
>>> CMake and configure<br>
>>><br>
>>> On Tue, Jul 31, 2012 at 12:24:33PM -0400, Justin Holewinski wrote:<br>
>>>><br>
>>>> On 07/31/2012 10:45 AM, Tom Stellard wrote:<br>
>>>>> On Mon, Jul 30, 2012 at 06:32:53PM -0400, Justin Holewinski wrote:<br>
>>>>>> On Mon, Jul 30, 2012 at 4:10 PM, Tom Stellard<br>
>>> <<a href="mailto:thomas.stellard@amd.com">thomas.stellard@amd.com</a>>wrote:<br>
>>>>>><br>
>>>>>>> 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<br>
>>> a<br>
>>>>>>>> more descriptive error message for when a staged back-end is<br>
>>> added<br>
>>>>>>>> to LLVM_TARGETS_TO_BUILD without setting LLVM_ENABLE_STAGING.<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-<br>
>>> targets option<br>
>>>>>>> to the configure script.<br>
>>>>>>><br>
>>>>>> Looks good to me, with the caveat that my autoconf-fu is pretty<br>
>>> weak.<br>
>>>>>><br>
>>>>>><br>
>>>>> Here are updated patches with s/STAGING/EXPERIMENTAL/.<br>
>>>>><br>
>>>>> -Tom<br>
>>>><br>
>>>> Looks good to me.<br>
>>>><br>
>>><br>
>>> Thanks.<br>
>>><br>
>>> Are there any autotool users that care to comment on the configure<br>
>>> patch?<br>
>>><br>
>>> -Tom<br>
>>><br>
>>>>><br>
>>>>><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.<br>
>>> 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<br>
>>> flag<br>
>>>>>>>>> doesn't actually do anything yet.<br>
>>>>>>>>><br>
>>>>>>>>> The idea for this patch comes from the LLVM staging area<br>
>>> discussion<br>
>>>>>>> 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-" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-</a><br>
>>> July/052146.html<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>
>>>>>>>> Thanks,<br>
>>>>>>>><br>
>>>>>>>> Justin Holewinski<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>
>>>>>><br>
>>>>>> --<br>
>>>>>><br>
>>>>>> Thanks,<br>
>>>>>><br>
>>>>>> Justin Holewinski<br>
>>>><br>
>>>> --<br>
>>>> Thanks,<br>
>>>><br>
>>>> Justin Holewinski<br>
>>>><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>
>> <add-staging-autoconf-option-v2.patch><add-staging-cmake-option-v3.patch>_______________________________________________<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>
><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>
> <experimental_targets.patch>_______________________________________________<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>
<br>
</div></div><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></blockquote></div><br></div>