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

Phabricator via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 14 13:36:09 PDT 2019


This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG376f64225288: [Bugpoint redesign] Reduced scope of variables in Delta implementation (authored by Diego Trevino Ferrer <diegof30 at gmail.com>).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66251/new/

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.215229.patch
Type: text/x-patch
Size: 824 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190814/5afbcd8a/attachment.bin>


More information about the llvm-commits mailing list