Re: [PATCH] winebuild: Use Clang to assemble if its integrated assembler is being used.

[André Hentschel]

Am 02.01.2013 02:56, schrieb Charles Davis:
> --- a/tools/winebuild/utils.c
> +++ b/tools/winebuild/utils.c
> @@ -345,10 +345,22 @@ struct strarray *get_as_command(void)
>  if (!as_command)
>  {
>
> +#ifdef CLANG_AS
> -
>    /* The native assembler might not support CFI pseudo-ops,
> -
>     * but clang does. So if we were compiled with
> -
>     * clang, we should assemble with clang, too.
> -
>     */
> -
>    static const char * const commands[] = { CLANG_AS, "clang", NULL };
> -
>    as_command = find_tool( "clang", commands );
>
> +#else
>          static const char * const commands[] = { "gas", "as", NULL };
>          as_command = find_tool( "as", commands );
> +#endif
>      }
>      strarray_add_one( args, as_command );
> +#ifdef CLANG_AS
> - strarray_add( args, "-xassembler", "-c", NULL );
>
> +#endif
>  if (force_pointer_size)
>  {

Not sure if you can do it like that, as i understand it winebuild should always be able to crosscompile something, these ifdefs would destroy that feature.