--wait-to-join / --no-wait-to-join option
authorKshitij Bansal <kshitij@cs.nyu.edu>
Wed, 3 Oct 2012 19:05:32 +0000 (19:05 +0000)
committerKshitij Bansal <kshitij@cs.nyu.edu>
Wed, 3 Oct 2012 19:05:32 +0000 (19:05 +0000)
workaround option till we fix bug 409. for now, I have kept --wait-to-join to
be default (old behavior). We could technically make --no-wait-to-join the
default when using non-incremental mode, but still possible for problems at
exit I think

(this commit was certified error- and warning-free by the test-and-commit script.)

src/main/command_executor_portfolio.cpp
src/main/options

index 4867082e69192c105b279b86c3d6013e7ac17c03..00527702e70643b4963e7101d0214f3f0a364002 100644 (file)
@@ -262,7 +262,8 @@ bool CommandExecutorPortfolio::doCommandSingleton(Command* cmd)
                          &d_smts[0]);
 
     pair<int, bool> portfolioReturn =
-        runPortfolio(d_numThreads, smFn, fns, true);
+        runPortfolio(d_numThreads, smFn, fns,
+                     d_options[options::waitToJoin]);
 
     d_seq = NULL;
     delete d_seq;
@@ -290,6 +291,7 @@ bool CommandExecutorPortfolio::doCommandSingleton(Command* cmd)
   } else {
     // Unreachable();
     assert(false);
+    return false;
   }
 
 }/* CommandExecutorPortfolio::doCommandSingleton() */
index 02c4643b346fd0f5f78e36ed8403217d376d151d..e78acf7d94bad0002f67ab04c6ce345c9ead4f46 100644 (file)
@@ -38,4 +38,7 @@ option separateOutput bool :default false
 option sharingFilterByLength --filter-lemma-length=N int :default -1 :read-write
  don't share lemmas strictly longer than N
 
+expert-option waitToJoin --wait-to-join bool :default true
+ wait for other threads to join before quitting
+
 endmodule