Instrumentation of events follows dependency on tape activity.

Author: jblueh

include/opdi/logic/omp/masterOmpLogic.cpp

@@ -54,12 +54,12 @@ void opdi::MasterOmpLogic::deleteFunc(void* dataPtr) {
 void opdi::MasterOmpLogic::onMaster(ScopeEndpoint endpoint) {
 
   #if OPDI_OMP_LOGIC_INSTRUMENT
-    for (auto& instrument : ompLogicInstruments) {
-      instrument->onMaster(endpoint);
-    }
-
     if (tool->getThreadLocalTape() != nullptr && tool->isActive(tool->getThreadLocalTape())) {
 
+        for (auto& instrument : ompLogicInstruments) {
+          instrument->onMaster(endpoint);
+        }
+
         Data* data = new Data;
         data->endpoint = endpoint;
 

include/opdi/logic/omp/syncRegionOmpLogic.cpp

@@ -82,13 +82,14 @@ bool opdi::SyncRegionOmpLogic::requiresReverseBarrier(SyncRegionKind kind, Scope
 
 void opdi::SyncRegionOmpLogic::onSyncRegion(SyncRegionKind kind, ScopeEndpoint endpoint) {
 
-  #if OPDI_OMP_LOGIC_INSTRUMENT
-    for (auto& instrument : ompLogicInstruments) {
-      instrument->onSyncRegion(kind, endpoint);
-    }
-  #endif
-
   if (tool->getThreadLocalTape() != nullptr && tool->isActive(tool->getThreadLocalTape())) {
+
+    #if OPDI_OMP_LOGIC_INSTRUMENT
+        for (auto& instrument : ompLogicInstruments) {
+          instrument->onSyncRegion(kind, endpoint);
+        }
+    #endif
+
     if (requiresReverseBarrier(kind, endpoint)) {
       internalPushHandle(kind, endpoint);
     }