[Backtracing] Add an option to output to stderr.

In CI, it would be better if the backtraces went to stderr rather than
stdout, so provide an option for that.

rdar://107192120

Author: al45tair

docs/Backtracing.rst

@@ -96,6 +96,10 @@ follows:
 |                 |         | only has effect on platforms that have a symbol  |
 |                 |         | cache that can be controlled by the runtime.     |
 +-----------------+---------+--------------------------------------------------+
+| output-to       | stdout  | Set to ``stderr`` to send the backtrace to the   |
+|                 |         | standard error instead of standard output.  This |
+|                 |         | may be useful in some CI systems.                |
++-----------------+---------+--------------------------------------------------+
 | swift-backtrace |         | If specified, gives the full path to the         |
 |                 |         | swift-backtrace binary to use for crashes.       |
 |                 |         | Otherwise, Swift will locate the binary relative |

include/swift/Runtime/Backtrace.h

@@ -97,6 +97,11 @@ enum class SanitizePaths {
   On = 1
 };
 
+enum class OutputTo {
+  Stdout = 0,
+  Stderr = 2,
+};
+
 struct BacktraceSettings {
   UnwindAlgorithm  algorithm;
   OnOffTty         enabled;
@@ -112,6 +117,7 @@ struct BacktraceSettings {
   SanitizePaths    sanitize;
   Preset           preset;
   bool             cache;
+  OutputTo         outputTo;
   const char      *swiftBacktracePath;
 };
 

stdlib/public/libexec/swift-backtrace/Target.swift

@@ -165,7 +165,7 @@ class Target {
     do {
       crashInfo = try reader.fetch(from: crashInfoAddr, as: CrashInfo.self)
     } catch {
-      print("swift-backtrace: unable to fetch crash info.")
+      print("swift-backtrace: unable to fetch crash info.", to: &standardError)
       exit(1)
     }
 
@@ -175,7 +175,7 @@ class Target {
 
     guard let mctx: MContext = try? reader.fetch(from: crashInfo.mctx,
                                                  as: MContext.self) else {
-      print("swift-backtrace: unable to fetch mcontext.")
+      print("swift-backtrace: unable to fetch mcontext.", to: &standardError)
       exit(1)
     }
 
@@ -195,12 +195,13 @@ class Target {
                           &threadCount)
 
     if kr != KERN_SUCCESS {
-      print("swift-backtrace: failed to enumerate threads - \(kr)")
+      print("swift-backtrace: failed to enumerate threads - \(kr)",
+            to: &standardError)
       exit(1)
     }
 
     guard let ports = threadPorts else {
-      print("swift-backtrace: thread array is nil")
+      print("swift-backtrace: thread array is nil", to: &standardError)
       exit(1)
     }
 
@@ -209,12 +210,13 @@ class Target {
       var kr = mach_thread_info(ports[Int(ndx)], THREAD_IDENTIFIER_INFO,
                                 &threadIdInfo)
       if kr != KERN_SUCCESS {
-        print("swift-backtrace: unable to get thread info for thread \(ndx) - \(kr)")
+        print("swift-backtrace: unable to get thread info for thread \(ndx) - \(kr)",
+              to: &standardError)
         exit(1)
       }
 
       guard let info = threadIdInfo else {
-        print("swift-backtrace: thread info is nil")
+        print("swift-backtrace: thread info is nil", to: &standardError)
         exit(1)
       }
 
@@ -228,7 +230,8 @@ class Target {
       if kr == KERN_SUCCESS {
         threadName = extInfo.pth_swiftName
       } else {
-        print("unable to fetch ext info \(kr)")
+        print("swift-backtrace: unable to fetch ext info \(kr)",
+              to: &standardError)
         threadName = ""
       }
 
@@ -250,14 +253,16 @@ class Target {
                                                    using: reader,
                                                    limit: limit,
                                                    top: top) else {
-        print("unable to capture backtrace from context for thread \(ndx)")
+        print("swift-backtrace: unable to capture backtrace from context for thread \(ndx)",
+              to: &standardError)
         exit(1)
       }
 
       guard let symbolicated = backtrace.symbolicated(with: images,
                                                       sharedCacheInfo: sharedCacheInfo,
                                                       useSymbolCache: cache) else {
-        print("unable to symbolicate backtrace from context for thread \(ndx)")
+        print("unable to symbolicate backtrace from context for thread \(ndx)",
+              to: &standardError)
         exit(1)
       }
 
@@ -280,14 +285,16 @@ class Target {
                                                    using: reader,
                                                    limit: limit,
                                                    top: top) else {
-        print("unable to capture backtrace from context for thread \(ndx)")
+        print("swift-backtrace: unable to capture backtrace from context for thread \(ndx)",
+              to: &standardError)
         continue
       }
 
       guard let symbolicated = backtrace.symbolicated(with: images,
                                                       sharedCacheInfo: sharedCacheInfo,
                                                       useSymbolCache: cache) else {
-        print("unable to symbolicate backtrace from context for thread \(ndx)")
+        print("swift-backtrace: unable to symbolicate backtrace from context for thread \(ndx)",
+              to: &standardError)
         continue
       }
 

stdlib/public/libexec/swift-backtrace/Utils.swift

@@ -139,4 +139,19 @@ internal func spawn(_ path: String, args: [String]) throws {
   }
 }
 
+struct CFileStream: TextOutputStream {
+  var fp: UnsafeMutablePointer<FILE>
+
+  public func write(_ string: String) {
+    fputs(string, fp)
+  }
+
+  public func flush() {
+    fflush(fp)
+  }
+}
+
+var standardOutput = CFileStream(fp: stdout)
+var standardError = CFileStream(fp: stderr)
+
 #endif // os(macOS)