Stability improvements (#26)

* omnitrace verbprintf and errprintf

* avail categories fix

* omnitrace-dl namespace

* OMNITRACE_CI macro / OMNITRACE_BUILD_CI option

- always enables asserts

* Roctracer improvements

- Reworked roctracer significantly
- Added categories to settings
- create_cpu_cid_entry
- handle clock_skew in roctracer
- fixed roctracer activity names
- hip_api_callback is "host"
- perfetto::Flow for GPU

* timemory submodule update

* Tweak to redirect

* Improved recursive guards

- functors component
- created "_hidden" variants of instrumentation funcs
  - omnitrace_* calls omnitrace_*_hidden
  - omnitrace-dl calls non-hidden
- omnitrace-dl now strongly protects against recursion
- omnitrace-dl now is standalone w.r.t. headers

* Stability fixes
- OMNITRACE_DEBUG_PUSH env variable
- fix to HSA_TOOLS_LIB in dl.cpp
- Fixed SFINAE warning in mpi_gotcha
- Handle 64, _l, _r extensions in whole function names

* cmake formatting

* Fix for last commit + push/pop count info

- don't instrument rocr::core::Signal::WaitAny
- don't instrument rocr::core::Runtime::AsyncEventsLoop
- fixed main not being popped in runtime instrument
- updated interval data reserve
- copy hash-ids and aliases onto main thread
- warn about unclosed regions
- removed guards in libomnitrace
- added error checks for incorrect push_count vs. pop_count
- fixed missing pop_timemory in last commit

* Finalization methodology updates

- added some more rocr:: functions to whole function names

* Add event_base_loop to whole functions

* Update VERSION to 0.1.0

[ROCm/rocprofiler-systems commit: 0d5c557552]

projects/rocprofiler-systems/source/lib/include/library/api.hpp

@@ -30,27 +30,34 @@
 extern "C"
 {
     /// handles configuration logic
-    void omnitrace_init_library(void) OMNITRACE_VISIBILITY("default");
+    void omnitrace_init_library(void) OMNITRACE_PUBLIC_API;
 
     /// starts gotcha wrappers
-    void omnitrace_init(const char*, bool, const char*) OMNITRACE_VISIBILITY("default");
+    void omnitrace_init(const char*, bool, const char*) OMNITRACE_PUBLIC_API;
 
     /// shuts down all tooling and generates output
-    void omnitrace_finalize(void) OMNITRACE_VISIBILITY("default");
+    void omnitrace_finalize(void) OMNITRACE_PUBLIC_API;
 
     /// sets an environment variable
-    void omnitrace_set_env(const char* env_name, const char* env_val)
-        OMNITRACE_VISIBILITY("default");
+    void omnitrace_set_env(const char* env_name,
+                           const char* env_val) OMNITRACE_PUBLIC_API;
 
     /// sets whether MPI should be used
-    void omnitrace_set_mpi(bool use, bool attached) OMNITRACE_VISIBILITY("default");
+    void omnitrace_set_mpi(bool use, bool attached) OMNITRACE_PUBLIC_API;
 
     /// starts an instrumentation region
-    void omnitrace_push_trace(const char* name) OMNITRACE_VISIBILITY("default");
+    void omnitrace_push_trace(const char* name) OMNITRACE_PUBLIC_API;
 
     /// stops an instrumentation region
-    void omnitrace_pop_trace(const char* name) OMNITRACE_VISIBILITY("default");
+    void omnitrace_pop_trace(const char* name) OMNITRACE_PUBLIC_API;
 
-    /// used by omnitrace-critical-trace
-    bool omnitrace_init_tooling() OMNITRACE_VISIBILITY("hidden");
+    // these are the real implementations for internal calling convention
+    void omnitrace_init_library_hidden(void) OMNITRACE_HIDDEN_API;
+    void omnitrace_init_hidden(const char*, bool, const char*) OMNITRACE_HIDDEN_API;
+    void omnitrace_finalize_hidden(void) OMNITRACE_HIDDEN_API;
+    void omnitrace_set_env_hidden(const char* env_name,
+                                  const char* env_val) OMNITRACE_HIDDEN_API;
+    void omnitrace_set_mpi_hidden(bool use, bool attached) OMNITRACE_HIDDEN_API;
+    void omnitrace_push_trace_hidden(const char* name) OMNITRACE_HIDDEN_API;
+    void omnitrace_pop_trace_hidden(const char* name) OMNITRACE_HIDDEN_API;
 }