Consolidate guid definitions (#112826)

* Consolidate guid definitions

* Replace minipal_guid_t with GUID

Author: am11

src/coreclr/debug/di/hash.cpp

@@ -201,7 +201,7 @@ void CordbHashTableEnum::BuildOrThrow(
     CordbBase * pOwnerObj,
     NeuterList * pOwnerList,
     CordbHashTable *pTable,
-    const _GUID &id,
+    const GUID &id,
     RSInitHolder<CordbHashTableEnum> * pHolder)
 {
     CordbHashTableEnum * pEnum = new CordbHashTableEnum(pOwnerObj, pOwnerList, pTable, id);

src/coreclr/debug/di/rspriv.h

@@ -1837,14 +1837,14 @@ public ICorDebugAssemblyEnum
         CordbBase * pOwnerObj,
         NeuterList * pOwnerList,
         CordbHashTable *table,
-        const _GUID &id);
+        const GUID &id);
 
 public:
     static void BuildOrThrow(
         CordbBase * pOwnerObj,
         NeuterList * pOwnerList,
         CordbHashTable *table,
-        const _GUID &id,
+        const GUID &id,
         RSInitHolder<CordbHashTableEnum> * pHolder);
 
     CordbHashTableEnum(CordbHashTableEnum *cloneSrc);

src/coreclr/debug/inc/dbgtransportsession.h

@@ -521,7 +521,7 @@ class DbgTransportSession
     // Struct defining the format of the data block sent with a SessionRequest.
     struct SessionRequestData
     {
-        minipal_guid_t  m_sSessionID;   // Unique session ID. Treated as byte blob so no endian-ness
+        GUID  m_sSessionID;   // Unique session ID. Treated as byte blob so no endian-ness
     };
 
     // Struct used to track a message that is being (or will soon be) sent but has not yet been acknowledged.
@@ -676,7 +676,7 @@ class DbgTransportSession
     // Session ID randomly allocated by the right side and sent over in the SessionRequest message. This
     // serves to disambiguate a re-send of the SessionRequest due to a network error versus a SessionRequest
     // from a different debugger.
-    minipal_guid_t  m_sSessionID;
+    GUID  m_sSessionID;
 
     // Lock used to synchronize sending messages and updating the session state. This ensures message bytes
     // don't become interleaved on the transport connection, the send queue is updated consistently across

src/coreclr/ilasm/portable_pdb.cpp

@@ -93,7 +93,7 @@ HRESULT PortablePdbWriter::Init(IMetaDataDispenserEx2* mdDispenser)
     time_t now;
     time(&now);
     m_pdbStream.id.pdbTimeStamp = (ULONG)now;
-    if (!minipal_guid_v4_create(reinterpret_cast<minipal_guid_t*>(&m_pdbStream.id.pdbGuid)))
+    if (!minipal_guid_v4_create(&m_pdbStream.id.pdbGuid))
     {
         hr = E_FAIL;
         goto exit;

src/coreclr/ildasm/dasm.cpp

@@ -5157,10 +5157,10 @@ void DumpCodeManager(IMAGE_COR20_HEADER *CORHeader, void* GUICookie)
     ULONG iCount = VAL32(CORHeader->CodeManagerTable.Size) / sizeof(GUID);
     for (ULONG i=0;  i<iCount;  i++)
     {
-        CHAR         rcguid[GUID_STR_BUFFER_LEN];
+        CHAR         rcguid[MINIPAL_GUID_BUFFER_LEN];
         GUID         Guid = *pcm;
         SwapGuid(&Guid);
-        GuidToLPSTR(Guid, rcguid);
+        minipal_guid_as_string(Guid, rcguid, MINIPAL_GUID_BUFFER_LEN);
         sprintf_s(szString,SZSTRING_SIZE,"//   [0x%08x]    %s", i, rcguid);
         printLine(GUICookie,szStr);
         pcm++;

src/coreclr/ildasm/dis.cpp

@@ -819,7 +819,7 @@ BOOL SourceLinesHelper(void *GUICookie, LineCodeDescr* pLCD, _Out_writes_(nSize)
 
     PAL_TRY(Param *, pParam, &param) {
         GUID guidLang={0},guidLangVendor={0},guidDoc={0};
-        CHAR zLang[GUID_STR_BUFFER_LEN],zVendor[GUID_STR_BUFFER_LEN],zDoc[GUID_STR_BUFFER_LEN];
+        CHAR zLang[MINIPAL_GUID_BUFFER_LEN],zVendor[MINIPAL_GUID_BUFFER_LEN],zDoc[MINIPAL_GUID_BUFFER_LEN];
         ULONG32 k;
         if(pParam->pLCD->FileToken != ulWasFileToken)
         {
@@ -830,9 +830,9 @@ BOOL SourceLinesHelper(void *GUICookie, LineCodeDescr* pLCD, _Out_writes_(nSize)
                 ||memcmp(&guidLangVendor,&guidWasLangVendor,sizeof(GUID))
                 ||memcmp(&guidDoc,&guidWasDoc,sizeof(GUID)))
             {
-                GuidToLPSTR(guidLang,zLang);
-                GuidToLPSTR(guidLangVendor,zVendor);
-                GuidToLPSTR(guidDoc,zDoc);
+                minipal_guid_as_string(guidLang, zLang, MINIPAL_GUID_BUFFER_LEN);
+                minipal_guid_as_string(guidLangVendor, zVendor, MINIPAL_GUID_BUFFER_LEN);
+                minipal_guid_as_string(guidDoc, zDoc, MINIPAL_GUID_BUFFER_LEN);
                 sprintf_s(szString,SZSTRING_SIZE,"%s%s '%s', '%s', '%s'", g_szAsmCodeIndent,KEYWORD(".language"),
                     zLang,zVendor,zDoc);
                 printLine(pParam->GUICookie,szString);

src/coreclr/ildasm/dman.cpp

@@ -120,7 +120,7 @@ void DumpScope(void* GUICookie)
     mdModule mdm;
     GUID mvid;
     WCHAR scopeName[1024];
-    CHAR guidString[GUID_STR_BUFFER_LEN];
+    CHAR guidString[MINIPAL_GUID_BUFFER_LEN];
     memset(scopeName,0,1024*sizeof(WCHAR));
     if(SUCCEEDED(g_pPubImport->GetScopeProps( scopeName, 1024, NULL, &mvid))&& scopeName[0])
     {
@@ -133,7 +133,7 @@ void DumpScope(void* GUICookie)
             VDELETE(sz);
         }
         printLine(GUICookie,szString);
-        GuidToLPSTR(mvid, guidString);
+        minipal_guid_as_string(mvid, guidString, MINIPAL_GUID_BUFFER_LEN);
         sprintf_s(szString,SZSTRING_SIZE,COMMENT("%s// MVID: %s"),g_szAsmCodeIndent,guidString);
 
         printLine(GUICookie,szString);

src/coreclr/inc/jiteeversionguid.h

@@ -32,16 +32,10 @@
 //////////////////////////////////////////////////////////////////////////////////////////////////////////
 //
 
-#ifndef GUID_DEFINED
-typedef struct _GUID {
-    uint32_t   Data1;    // NOTE: diff from Win32, for LP64
-    uint16_t   Data2;
-    uint16_t   Data3;
-    uint8_t    Data4[ 8 ];
-} GUID;
-typedef const GUID *LPCGUID;
-#define GUID_DEFINED
-#endif // !GUID_DEFINED
+#ifndef JIT_EE_VERSIONING_GUID_H
+#define JIT_EE_VERSIONING_GUID_H
+
+#include <minipal/guid.h>
 
 constexpr GUID JITEEVersionIdentifier = { /* 4463d6ac-dfcb-4ab0-a941-c53b56089b7c */
     0x4463d6ac,
@@ -50,8 +44,4 @@ constexpr GUID JITEEVersionIdentifier = { /* 4463d6ac-dfcb-4ab0-a941-c53b56089b7
     {0xa9, 0x41, 0xc5, 0x3b, 0x56, 0x08, 0x9b, 0x7c}
   };
 
-//////////////////////////////////////////////////////////////////////////////////////////////////////////
-//
-// END JITEEVersionIdentifier
-//
-//////////////////////////////////////////////////////////////////////////////////////////////////////////
+#endif // JIT_EE_VERSIONING_GUID_H

src/coreclr/inc/utilcode.h

@@ -35,7 +35,8 @@ using std::nothrow;
 
 #include "contract.h"
 
-#include <minipal/utils.h>
+#include <stddef.h>
+#include <minipal/guid.h>
 #include <dn-u16.h>
 
 #include "clrnt.h"
@@ -149,14 +150,10 @@ typedef LPSTR   LPUTF8;
 #endif
 #endif
 
-#include <stddef.h> // for offsetof
-#include <minipal/utils.h>
-
 #define IS_DIGIT(ch) (((ch) >= W('0')) && ((ch) <= W('9')))
 #define DIGIT_TO_INT(ch) ((ch) - W('0'))
 #define INT_TO_DIGIT(i) ((WCHAR)(W('0') + (i)))
 
-
 // Helper will 4 byte align a value, rounding up.
 #define ALIGN4BYTE(val) (((val) + 3) & ~0x3)
 
@@ -3031,23 +3028,6 @@ class ConfigMethodSet
     BYTE m_inited;
 };
 
-// 38 characters + 1 null terminating.
-#define GUID_STR_BUFFER_LEN (ARRAY_SIZE("{12345678-1234-1234-1234-123456789abc}"))
-
-//*****************************************************************************
-// Convert a GUID into a pointer to a string
-//*****************************************************************************
-int GuidToLPSTR(
-    REFGUID guid,   // [IN] The GUID to convert.
-    LPSTR szGuid,   // [OUT] String into which the GUID is stored
-    DWORD cchGuid); // [IN] Size in chars of szGuid
-
-template<DWORD N>
-int GuidToLPSTR(REFGUID guid, CHAR (&s)[N])
-{
-    return GuidToLPSTR(guid, s, N);
-}
-
 //*****************************************************************************
 // Convert a pointer to a string into a GUID.
 //*****************************************************************************

src/coreclr/nativeaot/Runtime/inc/CommonTypes.h

@@ -53,12 +53,6 @@ typedef uint32_t ULONG;
 typedef int64_t LONGLONG;
 typedef uint8_t BYTE;
 typedef uint16_t UINT16;
-typedef struct _GUID {
-    uint32_t Data1;
-    uint16_t Data2;
-    uint16_t Data3;
-    uint8_t Data4[8];
-} GUID;
 #endif // FEATURE_EVENT_TRACE && !_INC_WINDOWS
 
 // Hijack funcs are not called, they are "returned to". And when done, they return to the actual caller.