From: Matt Beaumont-Gay <matthewbg@google.com>
Date: Mon, 14 Nov 2011 23:24:52 +0000 (+0000)
Subject: Don't track depfiles as result files which need to be cleaned up on failure.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e1ba5be96e57738ba0d6302cdb471e12107a3201;p=clang

Don't track depfiles as result files which need to be cleaned up on failure.

This is a partial revert of r143846. While cleaning up after a crash is
probably a good idea, we were also deleting .d files if the compilation failed
due to invalid input, which is not the desired behavior. The test is XFAIL'd
until the cleanup code can be reworked to do the right thing.

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

diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp
index 9faac7174a..6a85fc00b3 100644
--- a/lib/Driver/Tools.cpp
+++ b/lib/Driver/Tools.cpp
@@ -225,13 +225,11 @@ void Clang::AddPreprocessingOptions(Compilation &C,
       DepFile = Output.getFilename();
     } else if (Arg *MF = Args.getLastArg(options::OPT_MF)) {
       DepFile = MF->getValue(Args);
-      C.addResultFile(DepFile);
     } else if (A->getOption().matches(options::OPT_M) ||
                A->getOption().matches(options::OPT_MM)) {
       DepFile = "-";
     } else {
       DepFile = darwin::CC1::getDependencyFileName(Args, Inputs);
-      C.addResultFile(DepFile);
     }
     CmdArgs.push_back("-dependency-file");
     CmdArgs.push_back(DepFile);
diff --git a/test/Driver/crash-cleanup.c b/test/Driver/crash-cleanup.c
index 218fcc64c7..b994bd85a3 100644
--- a/test/Driver/crash-cleanup.c
+++ b/test/Driver/crash-cleanup.c
@@ -4,6 +4,6 @@
 // REQUIRES: shell
 // REQUIRES: crash-recovery
 
-// XFAIL: darwin
+// XFAIL: *
 
 #pragma clang __debug crash