#1942: use isActive in termination and inline

DARMA-tasking · Sep 22, 2022 · cb76538 · cb76538
1 parent 13c86c5
commit cb76538
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/vt/termination/term_state.cc b/src/vt/termination/term_state.cc
@@ -121,15 +121,14 @@ TermCounterType TermState::decrementDependency() {
   return --deps_;
 }
 
-bool TermState::readySubmitParent(bool const needs_active) const {
+bool TermState::readySubmitParent() const {
   vtAssert(
     num_children_ != uninitialized_destination, "Children must be valid"
   );
 
-  auto const ret = (epoch_active_ or not needs_active) and
-    recv_child_count_ == num_children_ and local_terminated_ and
-    submitted_wave_ == cur_wave_ - 1 and not term_detected_ and
-    deps_ == 0;
+  auto const ret = epoch_active_ and local_terminated_ and
+    deps_ == 0 and recv_child_count_ == num_children_ and
+    submitted_wave_ == cur_wave_ - 1 and not term_detected_;
 
   vt_debug_print(
     verbose, term,

diff --git a/src/vt/termination/term_state.h b/src/vt/termination/term_state.h
@@ -66,7 +66,7 @@ struct TermState : EpochDependency, EpochLabel {
   void notifyLocalTerminated(bool const terminated = true);
   void submitToParent(bool const is_root, bool const setup = false);
   void receiveContinueSignal(TermWaveType const& wave);
-  bool readySubmitParent(bool const needs_active = true) const;
+  bool readySubmitParent() const;
   EventCountType getRecvChildCount() const;
   EpochType getEpoch() const;
   TermWaveType getCurWave() const;
@@ -76,6 +76,8 @@ struct TermState : EpochDependency, EpochLabel {
   void incrementDependency();
   TermCounterType decrementDependency();
 
+  inline bool isActive() const { return epoch_active_; }
+
   TermState(
     EpochType const& in_epoch, bool const in_local_terminated, bool const active,
     NodeType const& children