Thanks Doug.  That wasn't at all clear from looking at this file.  It just 
looked like a jumble of entries.  Sorting it made it clear to me what was 
actually being included in the build.  I'm fine with changing it back.

Out of curiosity, does changing the order here functionally make any 
difference?  Doesn't the dependency analysis ultimately determine the order in 
which things are built?

Sent from my iPad

On Aug 9, 2011, at 6:47 AM, Douglas Gregor <[email protected]> wrote:

> 
> 
> Sent from my iPhone
> 
> On Aug 8, 2011, at 8:39 PM, Ted Kremenek <[email protected]> wrote:
> 
>> Author: kremenek
>> Date: Mon Aug  8 22:39:10 2011
>> New Revision: 137107
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=137107&view=rev
>> Log:
>> Sort CMakeLists.txt.
>> 
>> Modified:
>>   cfe/trunk/tools/CMakeLists.txt
>> 
>> Modified: cfe/trunk/tools/CMakeLists.txt
>> URL: 
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/CMakeLists.txt?rev=137107&r1=137106&r2=137107&view=diff
>> ==============================================================================
>> --- cfe/trunk/tools/CMakeLists.txt (original)
>> +++ cfe/trunk/tools/CMakeLists.txt Mon Aug  8 22:39:10 2011
>> @@ -1,5 +1,5 @@
>> -add_subdirectory(libclang)
>> -add_subdirectory(c-index-test)
>> add_subdirectory(arcmt-test)
>> add_subdirectory(c-arcmt-test)
>> +add_subdirectory(c-index-test)
>> add_subdirectory(driver)
>> +add_subdirectory(libclang)
> 
> We typically like to have libraries come before the executables that link 
> against them.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to