[Comgr] Switch type to const char to fix warning

char * can be safely cast to const char *, so this should not cause any
breakages for API users.

The const designation more accurately describes this argument, and fixes
warnings with the tests, which now use const char * arguments

Author: lamb-j

amd/comgr/include/amd_comgr.h.in

@@ -2525,7 +2525,7 @@ amd_comgr_status_t AMD_COMGR_API
 amd_comgr_map_name_expression_to_symbol_name(
     amd_comgr_data_t data,
     size_t *size,
-    char *name_expression,
+    const char *name_expression,
     char *symbol_name) AMD_COMGR_VERSION_2_6;
 
 /**

amd/comgr/src/comgr.cpp

@@ -2152,7 +2152,8 @@ amd_comgr_populate_name_expression_map(amd_comgr_data_t Data, size_t *Count) {
 amd_comgr_status_t AMD_COMGR_API
 // NOLINTNEXTLINE(readability-identifier-naming)
 amd_comgr_map_name_expression_to_symbol_name(amd_comgr_data_t Data,
-                                             size_t *Size, char *NameExpression,
+                                             size_t *Size,
+                                             const char *NameExpression,
                                              char *SymbolName) {
   DataObject *DataP = DataObject::convert(Data);
   if (!DataP || !DataP->Data ||