[LLVMdev] [compiler-rt] CMake bug in building ARM builtins library

[sgundapa]

Yes, the fix is set_source_file_properties(<arm .S files> PROPERTIES LANGUAGE C)?

I will try using your patch for running builtins tests in my local set up.

 

Thanks

--Sumanth G

 

On Tue, Jul 8, 2014 at 2:44 PM, sgundapa <sgundapa at codeaurora.org> wrote:

No. CMake has different way of treating the .S files
Refer to http://www.cmake.org/Wiki/CMake/Assembler

I have a patch which will make CMake treat the .S files as source files.

 

Let me guess, you should just call set_source_file_properties(<arm .S files> PROPERTIES LANGUAGE C)?

 

But, how I can run the unit tests.  I see from compiler-rt/test/CMakeLists.txt

# BlocksRuntime and builtins testsuites are not yet ported to lit.
# add_subdirectory(BlocksRuntime)
# add_subdirectory(builtins)

How do you run the unit tests for builtins?

 

Well, there is http://reviews.llvm.org/D4251 which adds support for running the tests under CTest, but it's not yet reviewed.

There's also a "test/builtins/Unit/test" script, but I doubt it will run out of the box.

 



--Sumanth G

-----Original Message-----
From: Renato Golin [mailto:renato.golin at linaro.org]
Sent: Tuesday, July 08, 2014 12:19 PM
To: Alexey Samsonov
Cc: sgundapa; LLVM Dev
Subject: Re: [LLVMdev] [compiler-rt] CMake bug in building ARM builtins library

On 8 July 2014 19:47, Alexey Samsonov <vonosmas at gmail.com> wrote:
> compiler-rt/lib/builtins/arm/*.S files are listed in arm_SOURCES
> variable, and therefore should make it into the builtins static
> library on ARM. Don't they?

I assume so... But I'm not an expert in CMake.





 

-- 

Alexey Samsonov
vonosmas at gmail.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140711/cb1ecabd/attachment.html>