I'm not sure the defined(__APPLE__) is required as Host.mm cannot compile on 
anything else than Apple platform anyway.

Le 8 janv. 2014 à 23:12, Greg Clayton <[email protected]> a écrit :

> Author: gclayton
> Date: Wed Jan  8 16:12:52 2014
> New Revision: 198800
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=198800&view=rev
> Log:
> Only check os version for POSIX_SPAWN_CLOEXEC_DEFAULT on desktop builds of 
> LLDB.
> 
> 
> Modified:
>    lldb/trunk/source/Host/macosx/Host.mm
> 
> Modified: lldb/trunk/source/Host/macosx/Host.mm
> URL: 
> http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Host/macosx/Host.mm?rev=198800&r1=198799&r2=198800&view=diff
> ==============================================================================
> --- lldb/trunk/source/Host/macosx/Host.mm (original)
> +++ lldb/trunk/source/Host/macosx/Host.mm Wed Jan  8 16:12:52 2014
> @@ -1335,7 +1335,7 @@ GetPosixspawnFlags (ProcessLaunchInfo &l
>         flags |= POSIX_SPAWN_SETPGROUP;
> 
> #ifdef POSIX_SPAWN_CLOEXEC_DEFAULT
> -#if defined (__APPLE__)
> +#if defined (__APPLE__) && (defined (__x86_64__) || defined (__i386__))
>     static LazyBool g_use_close_on_exec_flag = eLazyBoolCalculate;
>     if (g_use_close_on_exec_flag == eLazyBoolCalculate)
>     {
> 
> 
> _______________________________________________
> lldb-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

-- Jean-Daniel





_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to