Merge pull request #217 from BlueBrain/pr/fix_instrumentor_phase_end

Omar Awile · web-flow · commit 48791d3ba2ea · 2019-11-08T09:36:47.000+01:00
use phase name to close the profiling phase
diff --git a/coreneuron/nrniv/profiler_interface.h b/coreneuron/nrniv/profiler_interface.h
@@ -279,11 +279,12 @@ using InstrumentorImpl = detail::Instrumentor<
 
 namespace Instrumentor {
 struct phase {
-    phase(const char* name) {
-        detail::InstrumentorImpl::phase_begin(name);
+    const char * phase_name;
+    phase(const char* name) : phase_name(name) {
+        detail::InstrumentorImpl::phase_begin(phase_name);
     }
     ~phase() {
-        detail::InstrumentorImpl::phase_end("");
+        detail::InstrumentorImpl::phase_end(phase_name);
     }
 };
 

Original file line number	Diff line number	Diff line change
`@@ -279,11 +279,12 @@ using InstrumentorImpl = detail::Instrumentor<`
`279`	`279`
`280`	`280`	`namespace Instrumentor {`
`281`	`281`	`struct phase {`
`282`		`- phase(const char* name) {`
`283`		`- detail::InstrumentorImpl::phase_begin(name);`
	`282`	`+ const char * phase_name;`
	`283`	`+ phase(const char* name) : phase_name(name) {`
	`284`	`+ detail::InstrumentorImpl::phase_begin(phase_name);`
`284`	`285`	`}`
`285`	`286`	`~phase() {`
`286`		`- detail::InstrumentorImpl::phase_end("");`
	`287`	`+ detail::InstrumentorImpl::phase_end(phase_name);`
`287`	`288`	`}`
`288`	`289`	`};`
`289`	`290`