From: Daniel Dunbar <daniel@zuster.org>
Date: Wed, 18 Mar 2009 23:18:19 +0000 (+0000)
Subject: Driver: Forcibly disable pipe support until we can execute them, the
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=60ccc7677a5416df3a8f8bde79ab35f85cb66e14;p=clang

Driver: Forcibly disable pipe support until we can execute them, the
driver is functional without them.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67254 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Driver/Driver.cpp b/lib/Driver/Driver.cpp
index d1db63650c..b147bb3a41 100644
--- a/lib/Driver/Driver.cpp
+++ b/lib/Driver/Driver.cpp
@@ -608,6 +608,11 @@ void Driver::BuildJobs(Compilation &C) const {
   llvm::PrettyStackTraceString CrashInfo("Building compilation jobs");
   bool SaveTemps = C.getArgs().hasArg(options::OPT_save_temps);
   bool UsePipes = C.getArgs().hasArg(options::OPT_pipe);
+
+  // FIXME: Pipes are forcibly disabled until we support executing
+  // them.
+  if (!CCCPrintBindings)
+    UsePipes = false;
   
   // -save-temps inhibits pipes.
   if (SaveTemps && UsePipes) {
@@ -684,6 +689,13 @@ void Driver::BuildJobsForAction(Compilation &C,
                                 const char *LinkingOutput,
                                 InputInfo &Result) const {
   llvm::PrettyStackTraceString CrashInfo("Building compilation jobs for action");
+
+  bool UsePipes = C.getArgs().hasArg(options::OPT_pipe);
+  // FIXME: Pipes are forcibly disabled until we support executing
+  // them.
+  if (!CCCPrintBindings)
+    UsePipes = false;
+
   if (const InputAction *IA = dyn_cast<InputAction>(A)) {
     // FIXME: This is broken, linker inputs won't work here.
     assert(isa<PositionalArg>(IA->getInputArg()) && "FIXME: Linker inputs");
@@ -749,7 +761,7 @@ void Driver::BuildJobsForAction(Compilation &C,
     if (AtTopLevel) {
       if (isa<PreprocessJobAction>(A) && !C.getArgs().hasArg(options::OPT_o))
         OutputToPipe = true;
-    } else if (C.getArgs().hasArg(options::OPT_pipe))
+    } else if (UsePipes)
       OutputToPipe = true;
   }