[cfe-commits] r110008 - /cfe/trunk/lib/Driver/Driver.cpp

Daniel Dunbar daniel at zuster.org
Sun Aug 1 19:38:06 PDT 2010


Author: ddunbar
Date: Sun Aug  1 21:38:06 2010
New Revision: 110008

URL: http://llvm.org/viewvc/llvm-project?rev=110008&view=rev
Log:
Driver: Never try to use piped inputs.

Modified:
    cfe/trunk/lib/Driver/Driver.cpp

Modified: cfe/trunk/lib/Driver/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Driver.cpp?rev=110008&r1=110007&r2=110008&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Driver.cpp (original)
+++ cfe/trunk/lib/Driver/Driver.cpp Sun Aug  1 21:38:06 2010
@@ -1061,7 +1061,6 @@
   const Tool &T = SelectToolForJob(C, TC, JA, Inputs);
 
   // Only use pipes when there is exactly one input.
-  bool TryToUsePipeInput = Inputs->size() == 1 && T.acceptsPipedInput();
   InputInfoList InputInfos;
   for (ActionList::const_iterator it = Inputs->begin(), ie = Inputs->end();
        it != ie; ++it) {
@@ -1074,7 +1073,7 @@
       SubJobAtTopLevel = true;
 
     InputInfo II;
-    BuildJobsForAction(C, *it, TC, BoundArch, TryToUsePipeInput,
+    BuildJobsForAction(C, *it, TC, BoundArch, false,
                        SubJobAtTopLevel, LinkingOutput, II);
     InputInfos.push_back(II);
   }
@@ -1094,11 +1093,7 @@
 
   // Figure out where to put the job (pipes).
   Job *Dest = &C.getJobs();
-  if (InputInfos[0].isPipe()) {
-    assert(TryToUsePipeInput && "Unrequested pipe!");
-    assert(InputInfos.size() == 1 && "Unexpected pipe with multiple inputs.");
-    Dest = &InputInfos[0].getPipe();
-  }
+  assert(!InputInfos[0].isPipe() && "Unrequested pipe!");
 
   // Always use the first input as the base input.
   const char *BaseInput = InputInfos[0].getBaseInput();





More information about the cfe-commits mailing list