#2302: Fix preprocessor directives

DARMA-tasking · Aug 5, 2024 · 5c60934 · 5c60934
1 parent 30b1083
commit 5c60934
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/src/vt/context/context.h b/src/vt/context/context.h
@@ -46,9 +46,6 @@
 
 #include <memory>
 #include <mpi.h>
-#if vt_check_enabled(papi)
-#include <papi.h>
-#endif
 
 #include "vt/config.h"
 #include "vt/runtime/component/component_pack.h"
@@ -65,6 +62,9 @@ struct RunnableNew;
 #if vt_check_enabled(trace_enabled)
 # include "vt/trace/trace_common.h"
 #endif
+#if vt_check_enabled(papi)
+# include <papi.h>
+#endif
 
 namespace vt {  namespace ctx {
 

diff --git a/src/vt/context/runnable_context/lb_data.impl.h b/src/vt/context/runnable_context/lb_data.impl.h
@@ -60,7 +60,7 @@ template <typename ElmT, typename MsgT>
 LBData::LBData(ElmT* in_elm, MsgT* msg)
   : lb_data_(&in_elm->getLBData()),
     cur_elm_id_(in_elm->getElmID()),
-    should_instrument_(msg->lbLiteInstrument()),
+    should_instrument_(msg->lbLiteInstrument())
 {
 #if vt_check_enabled(papi)
   papiData_ = std::make_unique<PAPIData>();

diff --git a/src/vt/runnable/runnable.cc b/src/vt/runnable/runnable.cc
@@ -199,6 +199,7 @@ void RunnableNew::run() {
 #endif
 }
 
+#if vt_check_enabled(papi)
 std::unordered_map<std::string, uint64_t> RunnableNew::getPAPIMetrics() {
   std::unordered_map<std::string, uint64_t> result = {};
   if (contexts_.has_lb)
@@ -207,6 +208,7 @@ std::unordered_map<std::string, uint64_t> RunnableNew::getPAPIMetrics() {
   }
   return result;
 }
+#endif
 
 void RunnableNew::start(TimeType time) {
   contexts_.setcontext.start();

diff --git a/src/vt/runnable/runnable.h b/src/vt/runnable/runnable.h
@@ -321,6 +321,7 @@ struct RunnableNew {
    */
   BaseMsgType* getMsg() const { return msg_.get(); }
 
+#if vt_check_enabled(papi)
   /**
    * \brief Start PAPI metrics map for the running context
    */
@@ -340,6 +341,7 @@ struct RunnableNew {
    * \return the dictionnary
    */
   std::unordered_map<std::string, uint64_t> getPAPIMetrics();
+#endif
 
 #if vt_check_enabled(fcontext)
   /**