[llvm-commits] CVS: llvm/lib/Support/SystemUtils.cpp

[Reid Spencer]

Changes in directory llvm/lib/Support:

SystemUtils.cpp updated: 1.38 -> 1.39
---
Log message:

Implement a function to print a warning if bytecode output is to be sent to
a terminal/console.


---
Diffs of the changes:  (+10 -8)

Index: llvm/lib/Support/SystemUtils.cpp
diff -u llvm/lib/Support/SystemUtils.cpp:1.38 llvm/lib/Support/SystemUtils.cpp:1.39
--- llvm/lib/Support/SystemUtils.cpp:1.38	Sun Dec 19 12:00:09 2004
+++ llvm/lib/Support/SystemUtils.cpp	Sat Jan  1 17:56:20 2005
@@ -14,20 +14,22 @@
 
 #include "llvm/Support/SystemUtils.h"
 #include "llvm/System/Program.h"
+#include "llvm/System/Process.h"
+#include <iostream>
 
 using namespace llvm;
 
-/// isStandardOutAConsole - Return true if we can tell that the standard output
-/// stream goes to a terminal window or console.
-bool llvm::isStandardOutAConsole() {
-#if HAVE_ISATTY
-  return isatty(1);
-#endif
-  // If we don't have isatty, just return false.
+bool llvm::CheckBytecodeOutputToConsole(std::ostream* stream_to_check) {
+  if (stream_to_check == &std::cout && sys::Process::StandardOutIsDisplayed()) {
+    std::cerr << "WARNING: You're attempting to print out a bytecode file.\n";
+    std::cerr << "This is inadvisable as it may cause display problems. If\n";
+    std::cerr << "you REALLY want to taste LLVM bytecode first-hand, you can\n";
+    std::cerr << "force output with the `-f' option.\n\n";
+    return true;
+  }
   return false;
 }
 
-
 /// FindExecutable - Find a named executable, giving the argv[0] of program
 /// being executed. This allows us to find another LLVM tool if it is built
 /// into the same directory, but that directory is neither the current