[cfe-commits] r67632 - /cfe/trunk/lib/Driver/Compilation.cpp

Daniel Dunbar daniel at zuster.org
Tue Mar 24 10:49:02 PDT 2009


Author: ddunbar
Date: Tue Mar 24 12:49:01 2009
New Revision: 67632

URL: http://llvm.org/viewvc/llvm-project?rev=67632&view=rev
Log:
Driver: Result files shouldn't be removed on failure when -save-temps
is specified.
 - No easy way to make a safe test case for this (given where the
   driver is supposed to put temp files).

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

Modified: cfe/trunk/lib/Driver/Compilation.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Compilation.cpp?rev=67632&r1=67631&r2=67632&view=diff

==============================================================================
--- cfe/trunk/lib/Driver/Compilation.cpp (original)
+++ cfe/trunk/lib/Driver/Compilation.cpp Tue Mar 24 12:49:01 2009
@@ -170,7 +170,7 @@
   CleanupFileList(TempFiles);
 
   // If the compilation failed, remove result files as well.
-  if (Res != 0)
+  if (Res != 0 && !getArgs().hasArg(options::OPT_save_temps))
     CleanupFileList(ResultFiles, true);
 
   return Res;





More information about the cfe-commits mailing list