[llvm-commits] [llvm] r81817 - in /llvm/trunk: lib/Target/ARM/AsmParser/ lib/Target/ARM/AsmParser/ARMAsmParser.cpp lib/Target/ARM/AsmParser/CMakeLists.txt lib/Target/ARM/AsmParser/Makefile lib/Target/ARM/Makefile test/MC/AsmParser/arm_word_directive.s test/MC/AsmParser/directive_values.s test/MC/AsmParser/x86_word_directive.s

[Kevin Enderby]

Hi Ted,

Thanks for fixing that.  I was trying to clean out my hacks of my  
wild copy and pastes and missed that one.  For now it shouldn't be  
there.

Kev

On Sep 14, 2009, at 9:06 PM, Ted Kremenek wrote:

>
> On Sep 14, 2009, at 5:27 PM, Kevin Enderby wrote:
>
>> ===================================================================== 
>> =========
>> --- llvm/trunk/lib/Target/ARM/AsmParser/CMakeLists.txt (added)
>> +++ llvm/trunk/lib/Target/ARM/AsmParser/CMakeLists.txt Mon Sep 14  
>> 19:27:25 2009
>> @@ -0,0 +1,6 @@
>> +include_directories( ${CMAKE_CURRENT_BINARY_DIR}/.. $ 
>> {CMAKE_CURRENT_SOURCE_DIR}/.. )
>> +
>> +add_llvm_library(LLVMARMAsmParser
>> +  ARMAsmParser.cpp
>> +  )
>> +add_dependencies(LLVMARMAsmParser)
>>
>
> Hi Kevin,
>
> This last line actually breaks the CMake builds:
>
>  CMake Error at lib/Target/ARM/AsmParser/CMakeLists.txt:6  
> (add_dependencies):
>    add_dependencies called with incorrect number of arguments
>
> Were you trying to add a specific dependency between the  
> LLVMARMAsmParser and something else?
>
> I've gone ahead and removed this last line, but I'm not certain if  
> that is the right thing to do.  The CMake build now works, but I'm  
> not certain if a dependency is suppose to be there.
>
> Ted
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090915/b1b057c5/attachment.html>