[PATCH] D51836: [bugpoint] Fix TempFile scoping bug

[Don Hinton via Phabricator via llvm-commits]

hintonda marked an inline comment as done.
hintonda added a comment.

This is a separate bug from the one you reported.

I have an idea on how to fix that one too, but since I don't have windows, I can't attempt a fix and test it.  However, I'll add a comment to PR38390 shortly.



================
Comment at: tools/bugpoint/ExecutionDriver.cpp:320
-    }
-  }
-
----------------
Meinersbur wrote:
> Is the error you suspect that `~DiscardTemp` of `Discard` is called here, before any work is done?
> 
> For http://llvm.org/PR38390, the file accessed is called `bugpoint-input-b43aded.bc` (`BugDriver::runPasses` creates file name with this pattern). The file deleted here would be called `bugpoint-test-program-....bc`.
Yes, if -save-temps is not passed, the file is deleted when Discard goes out of scope.


Repository:
  rL LLVM

https://reviews.llvm.org/D51836