[Bugpoint redesign] Reduced scope of variables in Delta implementation

[diegotf]

Summary

This diff also changed the check in Delta.cpp to verify interesting-ness, so it exits when the input isn't interesting

llvm/tools/llvm-reduce/deltas/Delta.cpp

 void llvm::runDeltaPass(
     TestRunner &Test, unsigned Targets,
     std::function<void(const std::vector<Chunk> &, Module *)>
         ExtractChunksFromModule) {
   if (!Targets) {
     outs() << "\nNothing to reduce\n";
     return;
   }
+  if (!Test.run(Test.getReducedFilepath())) {
+    outs() << "\nInput isn't interesting! Verify interesting-ness test\n";
+    exit(1);
+  }
 
   std::vector<Chunk> Chunks = {{1, Targets}};
   std::set<Chunk> UninterestingChunks;
   std::unique_ptr<Module> ReducedProgram;
 
-  if (!Test.run(Test.getReducedFilepath())) {
-    outs() << "\nInput isn't interesting! Verify interesting-ness test\n";
-    return;
-  }
-
   if (!increaseGranularity(Chunks)) {
     outs() << "\nAlready at minimum size. Cannot reduce anymore.\n";
     return;
   }
 
   do {
     UninterestingChunks = {};
     for (int I = Chunks.size() - 1; I >= 0; --I) {
void llvm::runDeltaPass(
 
   } while (!UninterestingChunks.empty() || increaseGranularity(Chunks));
 
   // If we reduced the testcase replace it
   if (ReducedProgram)
     Test.setProgram(std::move(ReducedProgram));
   outs() << "Couldn't increase anymore.\n";
 }
 No newline at end of file