Fix a PCH-related issue in the jitstdout change.
authorPat Gavlin <pagavlin@microsoft.com>
Fri, 13 May 2016 23:39:52 +0000 (16:39 -0700)
committerPat Gavlin <pagavlin@microsoft.com>
Fri, 13 May 2016 23:39:52 +0000 (16:39 -0700)
[tfs-changeset: 1604771]

src/jit/ee_il_dll.cpp
src/jit/host.h

index bf74ad0..a41b147 100644 (file)
@@ -13,10 +13,6 @@ XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
 XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
 */
 
-// This file is responsible for initializing `jitstdout` and therefore needs to
-// be able to refer to `stdout`.
-#define ALLOW_STDOUT
-
 #include "jitpch.h"
 #ifdef _MSC_VER
 #pragma hdrstop
@@ -72,11 +68,11 @@ void __stdcall jitStartup(ICorJitHost* jitHost)
     }
 
 #if defined(PLATFORM_UNIX)
-    jitstdout = stdout;
+    jitstdout = procstdout();
 #else
     if (jitstdout == nullptr)
     {
-        int jitstdoutFd = _dup(_fileno(stdout));
+        int jitstdoutFd = _dup(_fileno(procstdout()));
         _setmode(jitstdoutFd, _O_TEXT);
         jitstdout = _fdopen(jitstdoutFd, "w");
         assert(jitstdout != nullptr);
@@ -93,7 +89,7 @@ void jitShutdown()
 {
     Compiler::compShutdown();
 
-    if (jitstdout != stdout)
+    if (jitstdout != procstdout())
     {
         fclose(jitstdout);
     }
index bf1f60e..d38d41b 100644 (file)
@@ -54,10 +54,9 @@ const   size_t      OS_page_size = (4*1024);
 
 extern FILE* jitstdout;
 
-#if !defined(ALLOW_STDOUT)
+inline FILE* procstdout() { return stdout; }
 #undef stdout
 #define stdout use_jitstdout
-#endif
 
 /*****************************************************************************/
 #endif