From: Chris Bieneman Date: Thu, 19 Feb 2015 19:50:52 +0000 (+0000) Subject: Checking if TARGET_OS_IPHONE is defined isn't good enough for 10.7 and earlier. X-Git-Url: http://plrg.eecs.uci.edu/git/?p=oota-llvm.git;a=commitdiff_plain;h=cbccd8ddf555768ad957d03bc22c7e6a1f252fe2;hp=f45c1a4d12f065a249b897e35a7248e61ee12187;ds=sidebyside Checking if TARGET_OS_IPHONE is defined isn't good enough for 10.7 and earlier. Older versions of the TargetConditionals header always defined TARGET_OS_IPHONE to something (0 or 1), so we need to test not only for the existence but also if it is 1. This resolves PR22631. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@229904 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Support/Unix/Program.inc b/lib/Support/Unix/Program.inc index 2ed5597e4ad..baf2767ad58 100644 --- a/lib/Support/Unix/Program.inc +++ b/lib/Support/Unix/Program.inc @@ -42,10 +42,18 @@ #define _RESTRICT_KYWD #endif #include + #if defined(__APPLE__) #include #endif -#if !defined(__APPLE__) || defined(TARGET_OS_IPHONE) + +#if defined(__APPLE__) && !(defined(TARGET_OS_IPHONE) && TARGET_OS_IPHONE) +#define USE_NSGETENVIRON 1 +#else +#define USE_NSGETENVIRON 0 +#endif + +#if !USE_NSGETENVIRON extern char **environ; #else #include // _NSGetEnviron @@ -220,7 +228,7 @@ static bool Execute(ProcessInfo &PI, StringRef Program, const char **args, } if (!envp) -#if !defined(__APPLE__) || defined(TARGET_OS_IPHONE) +#if !USE_NSGETENVIRON envp = const_cast(environ); #else // environ is missing in dylibs.