[SDK] Release 1.0 Public API Modifications (#277)

* Make sure all structs/enums can be forward declared

* Updates to counter collection

- consistency updates and cleanup

* Conversion of dimension information to info struct

* Added deprecated folder

* Testing changes

* merge changes

* Fix shadowed variable

* Source code formatting

* Fix shadowed variable

* Update rocprofiler_counter_info_v1_t member names

* Split version.h into version.h and ext_version.h

- ext_version.h contains external version info, e.g. ROCPROFILER_HSA_API_TABLE_MAJOR_VERSION, ROCPROFILER_HSA_RUNTIME_VERSION
- this reduces amount of recompilation after a commit since version.h gets updated with the git revision

* profile_config -> counter_config

* EOF new line

* [Samples] Reduce header includes + reorg counter collection samples

* Misc compilation fixes

- shadowed variables
- use of [[deprecated("...")]] in C code
- unused variables

* Minor misc modifications

- use common:: instead of rocprofiler::common:: when inside rocprofiler namespace
- counters.cpp
  - move local anon namespace functions into rocprofiler::counters:: anon namespace
  - use std::string_view for get_static_string
  - const ref for get_static_ptr
  - misc namespace shortening

* [Public API] rocprofiler_get_version_triplet + rocprofiler_version_triplet_t

- struct rocprofiler_version_triplet_t containing fields for the major, minor, and patch version
- public API function: rocprofiler_get_version_triplet
- define C++ operators for rocprofiler_version_triplet_t
- C++ function compute_version_triplet

* [Tests] Improve async-copy-testing test

- relax constraints
- improve logging

* Update counter_config.h doxygen docs

* ROCPROFILER_SDK_BETA_COMPAT

- ppdef which helps with renaming when set to 1

* Remove spurious include

* Fix includes for cxx/version.hpp

* Doxygen fixes for rocprofiler_get_version and rocprofiler_get_version_triplet

* Public API Experimental Designation

- ROCPROFILER_SDK_EXPERIMENTAL added to experimental function
- "(experimental)" added to doxygen @brief entries

* Fix use of assert instead of static_assert in hip/stream.cpp

* Use typedef instead of define for rocprofiler_profile_config_id_t

* Use inline rocprofiler_{create,destroy}_profile_config instead of ppdef

- added <rocprofiler-sdk/deprecated/profile_config.h>

* Doxygen for rocprofiler_{create,destroy}_profile_config

* ROCPROFILER_SDK_DEPRECATED_WARNINGS

* Temporarily comment out ROCPROFILER_SDK_DEPRECATED_WARNINGS=1

* cmake formatting

* Misc variable renaming in samples and tests

* Fix declarations of types

* Fix hip stream tracing service struct name

- rocprofiler_callback_tracing_stream_handle_data_t renamed to rocprofiler_callback_tracing_hip_stream_api_data_t

* Rename "HIP_STREAM_API" to "HIP_STREAM"

---------

Co-authored-by: Jonathan R. Madsen <jonathanrmadsen@gmail.com>
Co-authored-by: Benjamin Welton <bewelton@amd.com>

[ROCm/rocprofiler-sdk commit: 4cd121e27b]

projects/rocprofiler-sdk/samples/api_callback_tracing/client.cpp

@@ -33,11 +33,10 @@
 
 #include "client.hpp"
 
-#include <rocprofiler-sdk/context.h>
-#include <rocprofiler-sdk/fwd.h>
-#include <rocprofiler-sdk/marker/api_id.h>
 #include <rocprofiler-sdk/registration.h>
 #include <rocprofiler-sdk/rocprofiler.h>
+#include <rocprofiler-sdk/cxx/operators.hpp>
+#include <rocprofiler-sdk/cxx/version.hpp>
 
 #include "common/call_stack.hpp"
 #include "common/defines.hpp"
@@ -318,7 +317,7 @@ stop()
 
 extern "C" rocprofiler_tool_configure_result_t*
 rocprofiler_configure(uint32_t                 version,
-                      const char*              runtime_version,
+                      const char*              version_string,
                       uint32_t                 priority,
                       rocprofiler_client_id_t* id)
 {
@@ -336,21 +335,26 @@ rocprofiler_configure(uint32_t                 version,
     // generate info string
     auto info = std::stringstream{};
     info << id->name << " (priority=" << priority << ") is using rocprofiler-sdk v" << major << "."
-         << minor << "." << patch << " (" << runtime_version << ")";
+         << minor << "." << patch << " (" << version_string << ")";
 
     std::clog << info.str() << std::endl;
 
     // demonstration of alternative way to get the version info
     {
-        auto version_info = std::array<uint32_t, 3>{};
-        ROCPROFILER_CALL(
-            rocprofiler_get_version(&version_info.at(0), &version_info.at(1), &version_info.at(2)),
-            "failed to get version info");
+        auto runtime_version = rocprofiler_version_triplet_t{};
+        ROCPROFILER_CALL(rocprofiler_get_version_triplet(&runtime_version),
+                         "failed to get version info");
 
-        if(std::array<uint32_t, 3>{major, minor, patch} != version_info)
+        if(rocprofiler_version_triplet_t{major, minor, patch} != runtime_version)
         {
             throw std::runtime_error{"version info mismatch"};
         }
+
+        if(rocprofiler_version_triplet_t{major, minor, patch} !=
+           rocprofiler::sdk::version::compute_version_triplet<100>(version))
+        {
+            throw std::runtime_error{"version triplet incorrectly calculated"};
+        }
     }
 
     // data passed around all the callbacks