Skip to content

Commit

Permalink
Actually test installed Papi logger when available
Browse files Browse the repository at this point in the history
  • Loading branch information
tcojean committed Jun 5, 2020
1 parent 36cb424 commit df7ae88
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 5 deletions.
2 changes: 1 addition & 1 deletion include/ginkgo/core/log/papi.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ class Papi : public Logger {
*/
static std::shared_ptr<Papi> create(
std::shared_ptr<const gko::Executor> exec,
const Logger::mask_type &enabled_events)
const Logger::mask_type &enabled_events = Logger::all_events_mask)
{
return std::shared_ptr<Papi>(new Papi(exec, enabled_events));
}
Expand Down
6 changes: 4 additions & 2 deletions test_install/test_install.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -196,10 +196,12 @@ int main(int, char **)
auto test = gko::log::Stream<>::create(refExec);
}

// core/log/convergence.hpp
#ifdef GKO_HAVE_PAPI_SDE
// core/log/papi.hpp
{
auto test = gko::log::Convergence<>::create(refExec);
auto test = gko::log::Papi<>::create(refExec);
}
#endif // GKO_HAVE_PAPI_SDE

// core/matrix/coo.hpp
{
Expand Down
6 changes: 4 additions & 2 deletions test_install/test_install_cuda.cu
Original file line number Diff line number Diff line change
Expand Up @@ -200,10 +200,12 @@ int main(int, char **)
gko::log::Stream<>::create(cudaExec);
}

// core/log/convergence.hpp
#ifdef GKO_HAVE_PAPI_SDE
// core/log/papi.hpp
{
gko::log::Convergence<>::create(cudaExec);
gko::log::Papi<>::create(cudaExec);
}
#endif // GKO_HAVE_PAPI_SDE

// core/matrix/coo.hpp
{
Expand Down

0 comments on commit df7ae88

Please sign in to comment.