[PATCH] D66251: [Bugpoint redesign] Reduced scope of variables in Delta implementation

[Diego Treviño via Phabricator via llvm-commits]

diegotf created this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

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


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D66251

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


Index: llvm/tools/llvm-reduce/deltas/Delta.cpp
===================================================================
--- llvm/tools/llvm-reduce/deltas/Delta.cpp
+++ llvm/tools/llvm-reduce/deltas/Delta.cpp
@@ -115,16 +115,15 @@
     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;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66251.215226.patch
Type: text/x-patch
Size: 824 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190814/7c539070/attachment.bin>