[Lldb-commits] [PATCH 3/4] cmake build: Fix building of HasAVX.s

[Jevin Sweval]

On Wed, Mar 12, 2014 at 3:43 PM, Kuba Ober <kuba at mareimbrium.org> wrote:
> ---
>  tools/debugserver/source/MacOSX/CMakeLists.txt | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/tools/debugserver/source/MacOSX/CMakeLists.txt b/tools/debugserver/source/MacOSX/CMakeLists.txt
> index b955917..8e2ec1c 100644
> --- a/tools/debugserver/source/MacOSX/CMakeLists.txt
> +++ b/tools/debugserver/source/MacOSX/CMakeLists.txt
> @@ -40,6 +40,12 @@ add_lldb_executable(debugserver
>    ${generated_mach_interfaces}
>    )
>
> +set_source_files_properties(
> +  HasAVX.s
> +  # Necessary since compilation will fail with stand-alone assembler
> +  PROPERTIES LANGUAGE C COMPILE_FLAGS "-x assembler-with-cpp"
> +  )
> +
>  add_dependencies(debugserver generate_dnbconfig)
>  target_link_libraries(debugserver ${DEBUGSERVER_USED_LIBS})
>

I noticed this one too. I tried to fix it "cleanly" by renaming it to
HasAVX.S (note the capital S) but CMake didn't get the hint that it
was supposed to be passed through cpp. Is this the cleanest way of
telling CMake what is going on?

Cheers,
Jevin